[PATCH] D61160: [SimplifyCFG] Implement the suggested ctlz transform
Shawn Landden via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Apr 28 19:57:49 PDT 2019
shawnl planned changes to this revision.
shawnl added a comment.
This transform is deeply flawed, and needs much work. If the default is not reachable, there is no need to limit to countPopulation == 1, only assure that there are no duplicates. If the default _is_ reachable, then countPopulation <= 1 continues to be important, but also ctlz and cttz must be run, except in very specific circumstances (such as utf-8 first character parsing)
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D61160/new/
https://reviews.llvm.org/D61160
More information about the llvm-commits
mailing list