diff options
| author | Hans Wennborg <hans@hanshq.net> | 2016-04-04 21:02:46 +0000 |
|---|---|---|
| committer | Hans Wennborg <hans@hanshq.net> | 2016-04-04 21:02:46 +0000 |
| commit | a47a692341bd108ad4188b6a868152d8d0f99b64 (patch) | |
| tree | fd290194bfb66b4c5fa6e9b17bc3fa03d3299c50 /llvm/lib/CodeGen/TailDuplication.cpp | |
| parent | 8cde42c40084bc62c3fbffbc2ed8b118813a008c (diff) | |
| download | bcm5719-llvm-a47a692341bd108ad4188b6a868152d8d0f99b64.tar.gz bcm5719-llvm-a47a692341bd108ad4188b6a868152d8d0f99b64.zip | |
Re-commit r265039 "[X86] Merge adjacent stack adjustments in eliminateCallFramePseudoInstr (PR27140)"
The original commit miscompiled things on 32-bit Windows, e.g. a Clang
boostrap. It turns out that mergeSPUpdates() was a bit too generous in
what it interpreted as a stack adjustment, causing the following code:
addl $12, %esp
leal -4(%ebp), %esp
To be "optimized" into simply:
addl $8, %esp
This commit tightens up mergeSPUpdates() and includes a new test
(test14 in movtopush.ll) for this situation.
llvm-svn: 265345
Diffstat (limited to 'llvm/lib/CodeGen/TailDuplication.cpp')
0 files changed, 0 insertions, 0 deletions
