diff options
| author | Evan Cheng <evan.cheng@apple.com> | 2010-04-15 18:42:28 +0000 |
|---|---|---|
| committer | Evan Cheng <evan.cheng@apple.com> | 2010-04-15 18:42:28 +0000 |
| commit | 1ba1428577a593612b80b6a7465a36d52b12cc70 (patch) | |
| tree | 1ec24b8011d7dfcc83b7138f2c18942bd3a795a8 /llvm/lib | |
| parent | 4230e35879bfa984a4910aac0baa5e3d11f6000a (diff) | |
| download | bcm5719-llvm-1ba1428577a593612b80b6a7465a36d52b12cc70.tar.gz bcm5719-llvm-1ba1428577a593612b80b6a7465a36d52b12cc70.zip | |
ARM SelectDYN_ALLOC should emit a copy from SP rather than referencing SP directly. In cases where there are two dyn_alloc in the same BB it would have caused the old SP value to be reused and badness ensues. rdar://7493908
llvm is generating poor code for dynamic alloca, I'll fix that later.
llvm-svn: 101383
Diffstat (limited to 'llvm/lib')
| -rw-r--r-- | llvm/lib/Target/ARM/ARMISelDAGToDAG.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Target/ARM/ARMISelDAGToDAG.cpp b/llvm/lib/Target/ARM/ARMISelDAGToDAG.cpp index 7d486631897..2f24d2ea1c6 100644 --- a/llvm/lib/Target/ARM/ARMISelDAGToDAG.cpp +++ b/llvm/lib/Target/ARM/ARMISelDAGToDAG.cpp @@ -945,7 +945,7 @@ SDNode *ARMDAGToDAGISel::SelectDYN_ALLOC(SDNode *N) { SDValue Chain = N->getOperand(0); SDValue Size = N->getOperand(1); SDValue Align = N->getOperand(2); - SDValue SP = CurDAG->getRegister(ARM::SP, MVT::i32); + SDValue SP = CurDAG->getCopyFromReg(Chain, dl, ARM::SP, MVT::i32); int32_t AlignVal = cast<ConstantSDNode>(Align)->getSExtValue(); if (AlignVal < 0) // We need to align the stack. Use Thumb1 tAND which is the only thumb |
