[PATCH] D83601: [ValueTracking] fix bug in maxnum case of cannotBeOrderedLessThanZeroImpl (PR46627)

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 13 15:20:10 PDT 2020


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

Patch updated:

1. Salvage a bit more of the existing functionality by differentiating based on "SignBitOnly" (shown with the unchanged test code comments).
2. Add another test (copysign) to show a miscompile.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D83601/new/

https://reviews.llvm.org/D83601

Files:
  llvm/lib/Analysis/ValueTracking.cpp
  llvm/test/Transforms/InstCombine/copysign.ll
  llvm/test/Transforms/InstSimplify/floating-point-arithmetic.ll
  llvm/test/Transforms/InstSimplify/floating-point-compare.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D83601.277587.patch
Type: text/x-patch
Size: 5918 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200713/26795851/attachment.bin>


More information about the llvm-commits mailing list