[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