[llvm] [AArch64] Combine and and lsl into ubfiz (PR #118974)
Cullen Rhodes via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 13 08:55:36 PST 2024
c-rhodes wrote:
> The codegen changes seem like an overall win to me. Nice! I had a few comments.
>
> Just a general question - did you try seeing if the SHL combine could be done in the general DAGCombiner to get the same effect? It seems like a sensible canonicalisation to do, mirroring what InstCombine does at the IR level. I just wondered if you saw regressions in other targets when doing that, or if something else got in the way? Thanks!
I didn't no, but I can take a look a that when I revisit this after holidays :)
https://github.com/llvm/llvm-project/pull/118974
More information about the llvm-commits
mailing list