[PATCH] D32605: Recognize CTLZ builtin

[Renato Golin via Phabricator via llvm-commits]

rengolin added a comment.

In https://reviews.llvm.org/D32605#740044, @joerg wrote:

> I agree. If the CPU has it, it will be beneficial. If it doesn't, it is only a useful transformation if the intrinsic can be constant folded.


@joerg, can you clarify what you agree with?

It sounds to be that you're worried that the intrinsics will be worse in some cases?

ARM(32/64) and x86_64 have CLZ, so it should always be beneficial, but I was just worries I'm missing something important.

cheers,
--renato


Repository:
  rL LLVM

https://reviews.llvm.org/D32605