[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