[PATCH] D151939: InstCombine: Recognize fneg(fabs) as bitcasted integer

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 31 16:09:09 PDT 2023


arsenm added a comment.

In D151939#4629115 <https://reviews.llvm.org/D151939#4629115>, @goldstein.w.n wrote:

> This doesn't really feel more canonical if we don't cast back.
> I'd argue if we don't cast back we are probably about to analyze some integer operations
> in which case `or` is more useful/recognizable than `fneg`/`fabs`.

If the integer was useful to downstream users, whatever it could have been should have happened already


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

https://reviews.llvm.org/D151939



More information about the llvm-commits mailing list