Juneyoung Lee 7583be01a6 [SimplifyCFG] Update FoldBranchToCommonDest to be poison-safe ... This patch makes FoldBranchToCommonDest merge branch conditions into `select i1` rather than `and/or i1` when it is called by SimplifyCFG. It is known that merging conditions into and/or is poison-unsafe, and this is towards making things *more* correct by removing possible miscompilations. Currently, InstCombine simply consumes these selects into and/or of i1 (which is also unsafe), so the visible effect would be very small. The unsafe select -> and/or transformation will be removed in the future. There has been efforts for updating optimizations to support the select form as well, and they are linked to D93065. The safe transformation is fired when it is called by SimplifyCFG only. This is done by setting the new `PoisonSafe` argument as true. Another place that calls FoldBranchToCommonDest is LoopSimplify. `PoisonSafe` flag is set to false in this case because enabling it has a nontrivial impact in performance because SCEV is more conservative with select form and InductiveRangeCheckElimination isn't aware of select form of and/or i1. Reviewed By: nikic Differential Revision: https://reviews.llvm.org/D95026		2021-03-08 01:38:03 +09:00
..
llvm	[SimplifyCFG] Update FoldBranchToCommonDest to be poison-safe	2021-03-08 01:38:03 +09:00
llvm-c	[DebugInfo] Add DIArgList MD to store multple values in DbgVariableIntrinsics	2021-03-05 17:02:24 +00:00