[PATCH] D141980: [X86] Transform slow LEA_B_I_D/LEA_SLOWBASE_I -> LEA_IS_D/LEA_IS iff base == index

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 18 09:50:01 PST 2023


craig.topper added a comment.

In D141980#4062735 <https://reviews.llvm.org/D141980#4062735>, @goldstein.w.n wrote:

> In D141980#4061628 <https://reviews.llvm.org/D141980#4061628>, @RKSimon wrote:
>
>> An LEA with scale > 1 also forces a slower computation on the AGUs for most AMD cpus
>
> I see, is there a flag you know of that you think should guard the transform?
>
> Although keep in mind, this is only hooked in for targets that have `TuningSlow3OpsLEA` which AFAICT only covers:
>
>   KNL
>   ICL
>   ICX
>   TGL
>   CNL
>   SKX
>   CLX
>   CPX
>   SKL
>   ADL
>   HSW
>   BDW
>   SNB
>   IVB
>
> Which are just the intel lineup. The only concerning one is `ADL` b.c. it has some `atom` cores and there where concerns `atom` has slow LEA_IS although looking at `ADL-E` on `uops.info` seems `LEA_IS_D` is same as `LEA_B_I_D` (2c latency each) so probably still fine to keep this transform (truthfully `ADL` probably shouldn't have `TuningSlow3OpsLEA` set as 2uops basically == 1x lea in worst case LEA on the E cores and the P cores have fast LEA).

I find it odd that KNL is in that list. KNL is based on Silvermont so I'd expect it to have `TuningSlowLEA` instead.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D141980/new/

https://reviews.llvm.org/D141980



More information about the llvm-commits mailing list