diff options
| author | Rafael Espindola <rafael.espindola@gmail.com> | 2009-06-21 12:02:51 +0000 |
|---|---|---|
| committer | Rafael Espindola <rafael.espindola@gmail.com> | 2009-06-21 12:02:51 +0000 |
| commit | 1c3329f7072356c8da84534ed0a7033b10f73062 (patch) | |
| tree | a87620155d085b90fd5ccc4ed02339d23b7c3c35 /lib/Target | |
| parent | 4577bbaa12be6d935163ff309978de221e483375 (diff) | |
| download | external_llvm-1c3329f7072356c8da84534ed0a7033b10f73062.zip external_llvm-1c3329f7072356c8da84534ed0a7033b10f73062.tar.gz external_llvm-1c3329f7072356c8da84534ed0a7033b10f73062.tar.bz2 | |
Fix PR4185.
Handle FpSET_ST0_80 being used when ST0 is still alive.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@73850 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Target')
| -rw-r--r-- | lib/Target/X86/X86FloatingPoint.cpp | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/lib/Target/X86/X86FloatingPoint.cpp b/lib/Target/X86/X86FloatingPoint.cpp index 0f2fbcc..ed4eb44 100644 --- a/lib/Target/X86/X86FloatingPoint.cpp +++ b/lib/Target/X86/X86FloatingPoint.cpp @@ -991,8 +991,13 @@ void FPS::handleSpecialFP(MachineBasicBlock::iterator &I) { case X86::FpSET_ST0_32: case X86::FpSET_ST0_64: case X86::FpSET_ST0_80: - assert((StackTop == 1 || StackTop == 2) - && "Stack should have one or two element on it to return!"); + // FpSET_ST0_80 is generated by copyRegToReg for both function return + // and inline assembly with the "st" constrain. In the latter case, + // it is possible for FP0 to be alive after this instruction. + if (!MI->killsRegister(X86::FP0)) { + // Duplicate ST0 + duplicateToTop(0, 0, I); + } --StackTop; // "Forget" we have something on the top of stack! break; case X86::FpSET_ST1_32: |