[PATCH] D151934: InstCombine: Recognize fneg when performed as bitcasted integer

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 8 15:34:50 PDT 2023


arsenm added a comment.

In D151934#4389126 <https://reviews.llvm.org/D151934#4389126>, @efriedma wrote:

> In any case, I'm more concerned about the UnsafeFPMath thing.

We already do this transform in the DAG, guarded by isFNegFree. I don't think this is a real issue. You can't really do anything unsafe with an fneg on its own, it has to be connected to something where the unsafe flag already applied. e.g. fadd (x, fneg(x)) -> 0 already had to have an nnan


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

https://reviews.llvm.org/D151934



More information about the llvm-commits mailing list