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

Noah Goldstein via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 30 12:03:39 PDT 2023


goldstein.w.n added a comment.

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`.

That being said it seems we don't any of the bitwise abs/neg ops even with the
complete roundtrip: https://godbolt.org/z/Wrje68hq1
and completely support patches to handle that.


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

https://reviews.llvm.org/D151939



More information about the llvm-commits mailing list