diff options
| author | Jim Grosbach <grosbach@apple.com> | 2010-02-22 23:10:38 +0000 |
|---|---|---|
| committer | Jim Grosbach <grosbach@apple.com> | 2010-02-22 23:10:38 +0000 |
| commit | 4642ad3af1cf508ac320b9afd25b065f08b36574 (patch) | |
| tree | c1dc05f9e433a2419c48dc5810f82f438367ce8a /lib/Target/ARM/ARMInstrThumb.td | |
| parent | 12f9168bc35da833097afdd551f3105b1d41a50f (diff) | |
| download | external_llvm-4642ad3af1cf508ac320b9afd25b065f08b36574.tar.gz external_llvm-4642ad3af1cf508ac320b9afd25b065f08b36574.tar.bz2 external_llvm-4642ad3af1cf508ac320b9afd25b065f08b36574.zip | |
Updated version of r96634 (which was reverted due to failing 176.gcc and
126.gcc nightly tests. These failures uncovered latent bugs that machine DCE
could remove one half of a stack adjust down/up pair, causing PEI to assert.
This update fixes that, and the tests now pass.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@96822 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Target/ARM/ARMInstrThumb.td')
| -rw-r--r-- | lib/Target/ARM/ARMInstrThumb.td | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/lib/Target/ARM/ARMInstrThumb.td b/lib/Target/ARM/ARMInstrThumb.td index b1f0459501..e8d3e228ec 100644 --- a/lib/Target/ARM/ARMInstrThumb.td +++ b/lib/Target/ARM/ARMInstrThumb.td @@ -120,7 +120,10 @@ def t_addrmode_sp : Operand<i32>, // Miscellaneous Instructions. // -let Defs = [SP], Uses = [SP] in { +// FIXME: Marking these as hasSideEffects is necessary to prevent machine DCE +// from removing one half of the matched pairs. That breaks PEI, which assumes +// these will always be in pairs, and asserts if it finds otherwise. Better way? +let Defs = [SP], Uses = [SP], hasSideEffects = 1 in { def tADJCALLSTACKUP : PseudoInst<(outs), (ins i32imm:$amt1, i32imm:$amt2), NoItinerary, "@ tADJCALLSTACKUP $amt1", |