[llvm] [ARM] Replace ABS and tABS machine nodes with custom lowering (PR #156717)
David Green via llvm-commits
llvm-commits at lists.llvm.org
Sun Sep 7 06:54:07 PDT 2025
https://github.com/davemgreen commented:
This sounds like a nice cleanup. I've wondered why the default lowering used these ABS pseudos. It is a shame it needs custom lowers, that can have downsides, but in this case with the custom lowering being so odd and the midend already canonicalizing to abs instructions it probably makes sense.
Can you update the costs to match the expansion?
https://github.com/llvm/llvm-project/pull/156717
More information about the llvm-commits
mailing list