[PATCH] D109958: [LoopFlatten] Enable it by default

Fri Dec 31 03:24:00 PST 2021

SjoerdMeijer added a comment.

In D109958#3214802 <https://reviews.llvm.org/D109958#3214802>, @nikic wrote:

> New compile-time: https://llvm-compile-time-tracker.com/compare.php?from=6f45fe9851c673883b3a258351ee4997aa2c028c&to=439d1c9613828db24ad03eb415baad2e76b8913c&stat=instructions Looks ok to me.

Nice, thank you for sharing!
That indeed looks good I think.

> We may want to drop the LoopSimplifyCFG run from the LegacyPM pipeline, as that doesn't match NewPM and seems to account for the actual codegen impact of this change (LoopFlatten itself doesn't seem to be doing much in practice).

I will look into that. I think I can do that separately.

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

https://reviews.llvm.org/D109958