[PATCH] D61726: [Pass Pipeline] Run another round of reassociation after loop pipeline

Tue May 14 07:05:53 PDT 2019

spatel added a comment.

In D61726#1497988 <https://reviews.llvm.org/D61726#1497988>, @nemanjai wrote:

> I have also run CTMark with and without the patch and it shows a minimal increase in compile time. This was run on a quiet PPC (Power9) machine set up for performance measurements with -j1:
>
>   Geomean difference                                                           0.3%

It would be interesting to see how that result translates on a more typical x86 build machine. Either way, I suspect we'll get different opinions about whether a 0.3% time increase is minimal and whether that cost is worth paying for the runtime perf gains. This might be a case for differentiating between -O2 and -O3?

Repository:
  rL LLVM

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

https://reviews.llvm.org/D61726