diff options
| author | Roman Lebedev <lebedev.ri@gmail.com> | 2020-10-04 11:27:07 +0300 |
|---|---|---|
| committer | Roman Lebedev <lebedev.ri@gmail.com> | 2020-10-04 11:53:50 +0300 |
| commit | 03bd5198b6f7d9f49d72e6516d813a206f3b6d0d (patch) | |
| tree | 446d3c38ce7cd3335b15690c62179df566ef2d87 /clang/lib/CodeGen/CodeGenFunction.cpp | |
| parent | 51beb0c80dc45d45efada5cd6e3a13c9fef0805c (diff) | |
| download | llvm-03bd5198b6f7d9f49d72e6516d813a206f3b6d0d.zip llvm-03bd5198b6f7d9f49d72e6516d813a206f3b6d0d.tar.gz llvm-03bd5198b6f7d9f49d72e6516d813a206f3b6d0d.tar.bz2 | |
[OldPM] Pass manager: run SROA after (simple) loop unrolling
I have stumbled into this pretty accidentally, when rewriting
some spaghetti-like code into something more structured,
which involved using some `std::array<>`s. And to my surprise,
the `alloca`s remained, causing about `+160%` perf regression.
https://llvm-compile-time-tracker.com/compare.php?from=bb6f4d32aac3eecb51909f4facc625219307ee68&to=d563e66f40f9d4d145cb2050e41cb961e2b37785&stat=instructions
suggests that this has geomean compile-time cost of `+0.08%`.
Note that D68593 / cecc0d27ad58c0aed8ef9ed99bbf691e137a0f26
already did this chage for NewPM, but left OldPM in a pessimized state.
This fixes [[ https://bugs.llvm.org/show_bug.cgi?id=40011 | PR40011 ]], [[ https://bugs.llvm.org/show_bug.cgi?id=42794 | PR42794 ]] and probably some other reports.
Reviewed By: nikic, xbolva00
Differential Revision: https://reviews.llvm.org/D87972
Diffstat (limited to 'clang/lib/CodeGen/CodeGenFunction.cpp')
0 files changed, 0 insertions, 0 deletions