[PATCH] D139785: [InstCombine] preserve signbit semantics of NAN with fold to fabs

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 11 11:03:16 PST 2022


spatel updated this revision to Diff 481932.
spatel edited the summary of this revision.
spatel added a comment.

Patch updated:
Adjusted code comment - we are implementing a model where the signbit is specified by fabs/fneg, so don't say it is meaningless.


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

https://reviews.llvm.org/D139785

Files:
  llvm/lib/Transforms/InstCombine/InstCombineSelect.cpp
  llvm/test/Transforms/InstCombine/fabs.ll
  llvm/test/Transforms/InstCombine/fneg-fabs.ll
  llvm/test/Transforms/InstCombine/fneg.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D139785.481932.patch
Type: text/x-patch
Size: 9801 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20221211/28e309dd/attachment.bin>


More information about the llvm-commits mailing list