[PATCH] D48122: [SimplifyCFG] Hoist common if-then-else code if used by two-entry PHI nodes

Alexandros Lamprineas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 13 08:50:34 PDT 2018


labrinea added a comment.

My argument would be that this change is fairly small. To be honest I don't know what is the status of GVN-Hoist. If it is in a good shape, then enabling it is the best way to go. If there are a few bugs to fix I am happy to help, but if it is far from being in a good shape then a workaround like this should be acceptable.


https://reviews.llvm.org/D48122





More information about the llvm-commits mailing list