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

[Elena Lepilkina via Phabricator via llvm-commits]

eklepilkina added a comment.

> 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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127726