[PATCH] D84108: [SimplifyCFG][LoopRotate] SimplifyCFG: disable common instruction hoisting by default, enable late in pipeline

Sat Jul 18 13:47:45 PDT 2020

lebedev.ri added a comment.

In D84108#2160494 <https://reviews.llvm.org/D84108#2160494>, @nikic wrote:

> > this does regress compile-time by +0.5% geomean (unsurprizingly)
>
> FWIW, a large part of the regressions seems to come down to a single file that regresses big time:
>
> > CMakeFiles/lencod.dir/context_ini.c.o 	3917M 	7067M (+80.41%)
>
> Probably worth taking a look whether that can be mitigated.

Thanks, i will take a look, just really wanted to finally post this after two dry-runs due to the rG0fdcca07ad2c0bdc2cdd40ba638109926f4f513b <https://reviews.llvm.org/rG0fdcca07ad2c0bdc2cdd40ba638109926f4f513b>/rG8d487668d09fb0e4e54f36207f07c1480ffabbfd <https://reviews.llvm.org/rG8d487668d09fb0e4e54f36207f07c1480ffabbfd> :)

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84108