[PATCH] D113939: [runtime-unroll] Remove restriction about unrolling multiple exit loops

[Philip Reames via Phabricator via llvm-commits]

reames added a comment.

Ran the test suite.  Surprisingly, this appears to be nearly performance neutral.

  tot-results  ...        diff

count  592.000000                                        ...  591.000000
mean   1472.037113                                       ...  0.000075  
std    13074.744904                                      ...  0.021348  
min    0.603100                                          ... -0.225786  
25%    2.796850                                          ... -0.000778  
50%    101.952583                                        ...  0.000079  
75%    471.058909                                        ...  0.001108  
max    217582.535333                                     ...  0.208422

I spot checked a couple of the worst regressions.  The assembly is unchanged, and the test is simply noisy as all heck.

On compile time, I would say the results are inconclusive, but don't show any major slowdown.

  tot-results  ...        diff

count  153.000000                                        ...  153.000000
mean   7.498642                                          ...  0.004975  
std    11.700351                                         ...  0.035278  
min    0.625100                                          ... -0.069157  
25%    1.598500                                          ... -0.010616  
50%    4.031300                                          ...  0.002407  
75%    6.612500                                          ...  0.017538  
max    92.241100                                         ...  0.202831

I will comment that I don't have much experience running test-suite, and ran this on an AWS instance.  There's definitely room for error in the run configuration to dominate measurements.  :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113939