[llvm-commits] CVS: llvm/lib/Target/X86/X86ISelLowering.cpp

[Evan Cheng]

Changes in directory llvm/lib/Target/X86:

X86ISelLowering.cpp updated: 1.44 -> 1.45
---
Log message:

Added a FIXME comment about why FST is currently flagged to fpGETRESULT.


---
Diffs of the changes:  (+6 -0)

 X86ISelLowering.cpp |    6 ++++++
 1 files changed, 6 insertions(+)


Index: llvm/lib/Target/X86/X86ISelLowering.cpp
diff -u llvm/lib/Target/X86/X86ISelLowering.cpp:1.44 llvm/lib/Target/X86/X86ISelLowering.cpp:1.45
--- llvm/lib/Target/X86/X86ISelLowering.cpp:1.44	Mon Jan 16 18:19:47 2006
+++ llvm/lib/Target/X86/X86ISelLowering.cpp	Mon Jan 16 18:37:42 2006
@@ -555,6 +555,9 @@
         Chain  = RetVal.getValue(1);
         InFlag = RetVal.getValue(2);
         if (X86ScalarSSE) {
+          // FIXME:Currently the FST is flagged to the FP_GET_RESULT. This
+          // shouldn't be necessary except for RFP cannot be live across
+          // multiple blocks. When stackifier is fixed, they can be uncoupled.
           unsigned Size = MVT::getSizeInBits(MVT::f64)/8;
           MachineFunction &MF = DAG.getMachineFunction();
           int SSFI = MF.getFrameInfo()->CreateStackObject(Size, Size);
@@ -1072,6 +1075,9 @@
         Chain  = RetVal.getValue(1);
         InFlag = RetVal.getValue(2);
         if (X86ScalarSSE) {
+          // FIXME:Currently the FST is flagged to the FP_GET_RESULT. This
+          // shouldn't be necessary except for RFP cannot be live across
+          // multiple blocks. When stackifier is fixed, they can be uncoupled.
           unsigned Size = MVT::getSizeInBits(MVT::f64)/8;
           MachineFunction &MF = DAG.getMachineFunction();
           int SSFI = MF.getFrameInfo()->CreateStackObject(Size, Size);