[llvm] [ARM] Replace ABS and tABS machine nodes with custom lowering (PR #156717)

via llvm-commits llvm-commits at lists.llvm.org
Sun Sep 7 15:45:27 PDT 2025


AZero13 wrote:

> 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?

Already done did you see? 

https://github.com/llvm/llvm-project/pull/156717


More information about the llvm-commits mailing list