[PATCH] D13076: [InstCombine] transform masking off of an FP sign bit into a fabs() intrinsic call (PR24886)

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 23 09:14:28 PDT 2015


spatel added a comment.

In http://reviews.llvm.org/D13076#251181, @majnemer wrote:

> If we will canonicalize away from bitmath, can we please teach `computeKnownBitsFromOperator` how to reason about `llvm.fabs`?  Otherwise, this might be a pessimization.


Sure, that seems fair. I see we already have that kind of logic for a few other intrinsics, so fabs should slide right in there.


http://reviews.llvm.org/D13076





More information about the llvm-commits mailing list