diff options
| author | Arnold Schwaighofer <aschwaighofer@apple.com> | 2015-05-08 23:52:00 +0000 |
|---|---|---|
| committer | Arnold Schwaighofer <aschwaighofer@apple.com> | 2015-05-08 23:52:00 +0000 |
| commit | f54b73d681d6d65a75cbd887939a75d652dbc24e (patch) | |
| tree | a7bcb3fe3cd39f0f19d4669de937fef7c42031e9 /llvm/test/CodeGen/ARM/ehabi.ll | |
| parent | 2ddfa0c5b2b8026cd4d6981da2cfd4ab03026d7c (diff) | |
| download | bcm5719-llvm-f54b73d681d6d65a75cbd887939a75d652dbc24e.tar.gz bcm5719-llvm-f54b73d681d6d65a75cbd887939a75d652dbc24e.zip | |
ScheduleDAGInstrs: In functions with tail calls PseudoSourceValues are not non-aliasing distinct objects
The code that builds the dependence graph assumes that two PseudoSourceValues
don't alias. In a tail calling function two FixedStackObjects might refer to the
same location. Worse 'immutable' fixed stack objects like function arguments are
not immutable and will be clobbered.
Change this so that a load from a FixedStackObject is not invariant in a tail
calling function and don't return a PseudoSourceValue for an instruction in tail
calling functions when building the dependence graph so that we handle function
arguments conservatively.
Fix for PR23459.
rdar://20740035
llvm-svn: 236916
Diffstat (limited to 'llvm/test/CodeGen/ARM/ehabi.ll')
| -rw-r--r-- | llvm/test/CodeGen/ARM/ehabi.ll | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/llvm/test/CodeGen/ARM/ehabi.ll b/llvm/test/CodeGen/ARM/ehabi.ll index ebf0c2a0033..088e48d2d79 100644 --- a/llvm/test/CodeGen/ARM/ehabi.ll +++ b/llvm/test/CodeGen/ARM/ehabi.ll @@ -146,8 +146,8 @@ declare void @_ZSt9terminatev() ; CHECK-FP: push {r4, r5, r6, r7, r8, r9, r10, r11, lr} ; CHECK-FP: .setfp r11, sp, #28 ; CHECK-FP: add r11, sp, #28 -; CHECK-FP: .pad #28 -; CHECK-FP: sub sp, sp, #28 +; CHECK-FP: .pad #44 +; CHECK-FP: sub sp, sp, #44 ; CHECK-FP: .personality __gxx_personality_v0 ; CHECK-FP: .handlerdata ; CHECK-FP: .fnend @@ -156,8 +156,8 @@ declare void @_ZSt9terminatev() ; CHECK-FP-ELIM: .fnstart ; CHECK-FP-ELIM: .save {r4, r5, r6, r7, r8, r9, r10, r11, lr} ; CHECK-FP-ELIM: push {r4, r5, r6, r7, r8, r9, r10, r11, lr} -; CHECK-FP-ELIM: .pad #28 -; CHECK-FP-ELIM: sub sp, sp, #28 +; CHECK-FP-ELIM: .pad #36 +; CHECK-FP-ELIM: sub sp, sp, #36 ; CHECK-FP-ELIM: .personality __gxx_personality_v0 ; CHECK-FP-ELIM: .handlerdata ; CHECK-FP-ELIM: .fnend @@ -205,7 +205,7 @@ declare void @_ZSt9terminatev() ; DWARF-FP: .cfi_offset r4, -36 ; DWARF-FP: add r11, sp, #28 ; DWARF-FP: .cfi_def_cfa r11, 8 -; DWARF-FP: sub sp, sp, #28 +; DWARF-FP: sub sp, sp, #44 ; DWARF-FP: sub sp, r11, #28 ; DWARF-FP: pop {r4, r5, r6, r7, r8, r9, r10, r11, lr} ; DWARF-FP: mov pc, lr @@ -226,9 +226,9 @@ declare void @_ZSt9terminatev() ; DWARF-FP-ELIM: .cfi_offset r6, -28 ; DWARF-FP-ELIM: .cfi_offset r5, -32 ; DWARF-FP-ELIM: .cfi_offset r4, -36 -; DWARF-FP-ELIM: sub sp, sp, #28 -; DWARF-FP-ELIM: .cfi_def_cfa_offset 64 -; DWARF-FP-ELIM: add sp, sp, #28 +; DWARF-FP-ELIM: sub sp, sp, #36 +; DWARF-FP-ELIM: .cfi_def_cfa_offset 72 +; DWARF-FP-ELIM: add sp, sp, #36 ; DWARF-FP-ELIM: pop {r4, r5, r6, r7, r8, r9, r10, r11, lr} ; DWARF-FP-ELIM: mov pc, lr ; DWARF-FP-ELIM: .cfi_endproc |
