[PATCH] D13222: [ValueTracking] teach computeKnownBits that a fabs() clears sign bits

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 8 11:02:35 PDT 2015


spatel added a comment.

In http://reviews.llvm.org/D13222#263038, @arsenm wrote:

> Should this also be done in DAGCombiner?


Sure - I would think that anything that's being done in ValueTracking should be done there too. Can this be refactored, so we're not duplicating code?


Repository:
  rL LLVM

http://reviews.llvm.org/D13222





More information about the llvm-commits mailing list