[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