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

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 13 11:18:53 PDT 2018


efriedma added a comment.

GVNHoist is generally pretty stable; my team has used it to build a lot of code without any issues.  There are a few known bugs (https://bugs.llvm.org/show_bug.cgi?id=37791, https://bugs.llvm.org/show_bug.cgi?id=36635, https://bugs.llvm.org/show_bug.cgi?id=37445 ) that we probably need to fix before turning it on by default.


https://reviews.llvm.org/D48122





More information about the llvm-commits mailing list