[PATCH] D83663: [LoopRotate] Bump rotation-max-header-size to 64

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.

> - there's pretty small (+0.01 .. +0.05% compile-time regression)
> - this causes, on average, +0.03% size-text increase So i'm wondering, what is the procedure here, would this be a reasonable change?

I think the cap is mostly there to guard against unnecessary code size increases.

It would be interesting to see what the impact on code size would be with -Os across more than the ctmark tests. I think for -Oz we already force the header size to 0, but I am not sure if it would be great to add another -Os threshold.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83663