[llvm-commits] two-byte return optimization for AMD

Evan Cheng echeng at apple.com
Sun Dec 7 23:04:03 PST 2008


I am not crazy about adding a subtarget feature for a micro- 
optimization. If you want to enable a family of optimizations for AMD  
chips, please just add something like a FeatureAMD feature.

Also, it seems awfully expensive to perform a full machine function  
pass. This seems like something that can be done at instruction  
selection time?

Evan

On Dec 7, 2008, at 12:19 AM, Nick Lewycky wrote:

> The AMD optimization manual suggests avoiding branches to 'ret'  
> instructions, preferring to emit 'rep; ret'. The attached patch  
> implements this.
>
> This doesn't apply very often across llvm-test. The largest is  
> kimwitu++ which produces 28 of these two-byte rets. The performance  
> impact seems to be less than noise, at least on my system.
>
> Please review!
>
> Nick
> <x86-twobyteret.patch>_______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits




More information about the llvm-commits mailing list