[PATCH] D37940: Peel off the dominant case in switch statement

Daniel Berlin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 15 16:35:38 PDT 2017


dberlin added a comment.

We've already discovered a slew of problem from over-optimizing things in simplifycfg, which is also being used as a canonicalization pass.
Either it needs a set of flags controlling what it does, or it needs to not be trying to do "performance improvement" and "canonicalization" as a single pass.
See, e.g, https://bugs.llvm.org/show_bug.cgi?id=34603


https://reviews.llvm.org/D37940





More information about the llvm-commits mailing list