[PATCH] D127726: [InstCombiner] Add option to replace PHI of GEPs with GEP with PHI as index

Mon Jul 18 04:02:10 PDT 2022

xbolva00 added a comment.

In D127726#3659169 <https://reviews.llvm.org/D127726#3659169>, @eklepilkina wrote:

>> Maybe you could take a look at one big regression in MicroBenchmarks/LCALS/SubsetALambdaLoops/lcalsALambda.test:BM_PRESSURE_CALC_LAMBDA/5001
>
> I had a look, I have already mentioned above that many of Microbenchmarks are very unstable. I ran several times for bothe versions and all results are in the same range for both cases. I compared binaries generated and they are the same for the binary that contains this microbenchmark.
>
> So I also need to mention that some improvements also can be the result of this unstability of benchmarks.

Ah, right. Thanks for info.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127726