[PATCH] D45173: [InstCombine] Recognize idioms for ctpop and ctlz

Krzysztof Parzyszek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 2 12:35:56 PDT 2018


kparzysz added a comment.

In https://reviews.llvm.org/D45173#1054694, @efriedma wrote:

> Does this trigger on the compiler-rt implementation of __popcountsi2/__clzsi2?  I guess it technically isn't a problem if it does, given the LLVM backend currently doesn't generate calls to them, but it might be worth adding a backend testcase to make sure we don't generate an infinite loop.


Would it make sense to check the function name for these two above and exit early if it matches?


Repository:
  rL LLVM

https://reviews.llvm.org/D45173





More information about the llvm-commits mailing list