[PATCH] D38133: [CGP] Make optimizeMemoryInst able to combine more kinds of ExtAddrMode fields

[Serguei Katkov via Phabricator via llvm-commits]

skatkov added a comment.

Hi John, in general looks good to me. I wonder whether we can add some options to enable/disable different fields and enable them step-by-step?
The patch has a significant impact and in this case people can use these options to quickly detect where the problem is. Does it make sense?

Please postpone the landing of this patch until I finish enabling the optimization itself.


Repository:
  rL LLVM

https://reviews.llvm.org/D38133