[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