[PATCH] D14590: [SimplifyLibCalls] Constant folding for fls, flsl, flsll

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 15 14:12:27 PST 2016


davide added a comment.

In https://reviews.llvm.org/D14590#624289, @efriedma wrote:

> That's weird... I would have guessed instcombine would kick in an canonicalize one way or the other.  Anyway, the difference between 1 and 3 doesn't really matter.


I'll go for 3, then. 
Slightly related, we seem to produce slightly worse code than gcc-4.2 for ffs(x), see https://godbolt.org/g/CruJtS
 I don't think it matters much, but still worth mentioning.


https://reviews.llvm.org/D14590





More information about the llvm-commits mailing list