[PATCH] D19391: transform obscured FP sign bit ops into a fabs/fneg using TLI hook

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 2 12:02:08 PDT 2016


spatel updated this revision to Diff 59440.
spatel added a comment.

Patch updated:
Revert r249701 ([ValueTracking] teach computeKnownBits that a fabs() clears sign bits).
The LangRef mentions IEEE format for NaN/Inf in the IR, but I'm not sure if that allows us to deduce the position of the sign bit in IR FP formats. To be safe, we'll just remove the fabs() ValueTracking logic for now.


http://reviews.llvm.org/D19391

Files:
  include/llvm/Target/TargetLowering.h
  lib/Analysis/ValueTracking.cpp
  lib/CodeGen/SelectionDAG/DAGCombiner.cpp
  lib/Target/AArch64/AArch64ISelLowering.h
  lib/Transforms/InstCombine/InstCombineAndOrXor.cpp
  test/CodeGen/AArch64/fcvt-int.ll
  test/Transforms/InstCombine/and2.ll
  test/Transforms/InstCombine/fabs.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D19391.59440.patch
Type: text/x-patch
Size: 10035 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160602/0dd70236/attachment.bin>


More information about the llvm-commits mailing list