[PATCH] D101727: Fix PR47960 - Incorrect transformation of fabs with nnan flag

[Krishna Kariya via Phabricator via llvm-commits]

Krishnakariya updated this revision to Diff 357039.
Krishnakariya marked an inline comment as done.
Krishnakariya added a comment.

Updates:

1. Added check for nnan (earlier removed) and added new tests.
2. Check for nsz in select for fneg logic is required. So, I didn't remove this check. Please see this example: https://alive2.llvm.org/ce/z/sEfTYx.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101727

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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D101727.357039.patch
Type: text/x-patch
Size: 25893 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210707/6f951b40/attachment.bin>