[PATCH] D35014: [X86] PR32755 : Improvement in CodeGen instruction selection for LEAs.

Jatin Bhateja via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 6 04:48:53 PDT 2017


jbhateja added a comment.

In https://reviews.llvm.org/D35014#861853, @lsaba wrote:

> 3-Ops LEA are costly starting target SandyBridge , is there a limitation in the code for the targets this transformation works on? If not I think there should be. 
>  you can check the Slow3OpsLEA feature for the full list of targets.


Yes, this check could be added in pattern matching where i'm avoiding creation of complex LEA with scale less than equal to 2.

Please let me know if anything else you see to save iteration.
Thanks


https://reviews.llvm.org/D35014





More information about the llvm-commits mailing list