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

via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 11 14:59:03 PDT 2025


AZero13 wrote:

> > Is 2 not accurate since it is just cmp+cmov?
> 
> Yeah It depends on how much a cmp costs. The BasicTTI seems to use ICmp+Select+Sub costs, even if the default expansion is via asr+xor+sub. The cost of i64 seems too low, and the costsize costs of the others you could argue should be higher than 2.

Well, ICmp+Select+Sub is the custom lowering we are going with. We are not doing that in THUMB. 

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


More information about the llvm-commits mailing list