diff options
| author | Nikita Popov <npopov@redhat.com> | 2023-03-14 15:26:41 +0100 |
|---|---|---|
| committer | Nikita Popov <npopov@redhat.com> | 2023-03-14 17:00:17 +0100 |
| commit | fb5683449e97bd8c2d107128dcf08d6f93789315 (patch) | |
| tree | 37c5a4ff093c3190bb3748b3229876471f8dc1a9 /libcxx/test/std/strings/basic.string/string.modifiers/string_append/iterator.pass.cpp | |
| parent | d9db5a5904fd328130a5dd100e3cf36eb7d9e6d0 (diff) | |
[Pipelines] Restore old DAE position in LTO pipeline
This is a partial revert of D128830, restoring the previous
position of DeadArgElim in the fat LTO pipeline. The motivation
for this is a major code size regression observed in Rust and
illustrated in the PhaseOrdering test.
This is a conservative fix restoring the previous pipeline order.
The real problem is that the LTO pipeline is conceptually broken:
It doesn't have a CGSCC function simplification pipeline. The
inliner is just being run by itself. This wouldn't be a problem
if fat LTO used a standard design where ArgPromotion and DAE are
only run after functions have already been simplified by the
CGSCC inliner pipeline.
Differential Revision: https://reviews.llvm.org/D146051
Diffstat (limited to 'libcxx/test/std/strings/basic.string/string.modifiers/string_append/iterator.pass.cpp')
0 files changed, 0 insertions, 0 deletions
