summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRafael Espindola <rafael.espindola@gmail.com>2009-06-29 20:29:59 +0000
committerRafael Espindola <rafael.espindola@gmail.com>2009-06-29 20:29:59 +0000
commit538064d6b1f210c06413784f04d0d892f1f84e53 (patch)
tree0e64a9a27a458ca213922b3eb90aeff1d7bffe76
parentf71ee01c81faf9a1070096cf1e5268c25998eaec (diff)
downloadbcm5719-llvm-538064d6b1f210c06413784f04d0d892f1f84e53.tar.gz
bcm5719-llvm-538064d6b1f210c06413784f04d0d892f1f84e53.zip
FIX PR 4459.
Not sure I understand how the temp register gets used, but this fixes a bug and introduces no regressions. llvm-svn: 74446
-rw-r--r--llvm/lib/Target/X86/X86FloatingPoint.cpp2
-rw-r--r--llvm/test/CodeGen/X86/inline-asm-fpstack3.ll15
2 files changed, 16 insertions, 1 deletions
diff --git a/llvm/lib/Target/X86/X86FloatingPoint.cpp b/llvm/lib/Target/X86/X86FloatingPoint.cpp
index ed4eb444e9c..abacdb2f9b6 100644
--- a/llvm/lib/Target/X86/X86FloatingPoint.cpp
+++ b/llvm/lib/Target/X86/X86FloatingPoint.cpp
@@ -996,7 +996,7 @@ void FPS::handleSpecialFP(MachineBasicBlock::iterator &I) {
// it is possible for FP0 to be alive after this instruction.
if (!MI->killsRegister(X86::FP0)) {
// Duplicate ST0
- duplicateToTop(0, 0, I);
+ duplicateToTop(0, 7 /*temp register*/, I);
}
--StackTop; // "Forget" we have something on the top of stack!
break;
diff --git a/llvm/test/CodeGen/X86/inline-asm-fpstack3.ll b/llvm/test/CodeGen/X86/inline-asm-fpstack3.ll
new file mode 100644
index 00000000000..ac89a1d9ad5
--- /dev/null
+++ b/llvm/test/CodeGen/X86/inline-asm-fpstack3.ll
@@ -0,0 +1,15 @@
+; RUN: llvm-as < %s | llc -march=x86 > %t
+; RUN: grep {fld %%st(0)} %t
+; PR4459
+
+declare x86_fp80 @ceil(x86_fp80)
+
+declare void @test(x86_fp80)
+
+define void @test2(x86_fp80 %a) {
+entry:
+ %0 = call x86_fp80 @ceil(x86_fp80 %a)
+ call void asm sideeffect "fistpl $0", "{st}"( x86_fp80 %0)
+ call void @test(x86_fp80 %0 )
+ ret void
+}
OpenPOWER on IntegriCloud