[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