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

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

Ping.

This patch gets us closer to the desired behavior of IEEE-compliant fneg/fabs/copysign (there is no "copy" in IR) semantics with the default LLVM FP environment, and the code comment is hopefully clear now.

If there are other open questions, those should be added to the discourse thread or in bug reports.


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

https://reviews.llvm.org/D139785