[llvm] 0851350 - [X86] Update stale comment. NFC

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 3 16:19:25 PDT 2020


Author: Craig Topper
Date: 2020-09-03T16:19:10-07:00
New Revision: 08513505571b0167f387c4e957ddcc138f77a9ca

URL: https://github.com/llvm/llvm-project/commit/08513505571b0167f387c4e957ddcc138f77a9ca
DIFF: https://github.com/llvm/llvm-project/commit/08513505571b0167f387c4e957ddcc138f77a9ca.diff

LOG: [X86] Update stale comment. NFC

The optimization in ExpandIntOp_UINT_TO_FP was removed in D72728
in January 2020.

Added: 
    

Modified: 
    llvm/lib/Target/X86/X86ISelLowering.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/X86/X86ISelLowering.cpp b/llvm/lib/Target/X86/X86ISelLowering.cpp
index 6560ab2bcccd..7a97527ecdf2 100644
--- a/llvm/lib/Target/X86/X86ISelLowering.cpp
+++ b/llvm/lib/Target/X86/X86ISelLowering.cpp
@@ -20149,10 +20149,8 @@ SDValue X86TargetLowering::LowerUINT_TO_FP(SDValue Op,
   SDValue Store =
       DAG.getStore(Chain, dl, ValueToStore, StackSlot, MPI, Align(8));
   // For i64 source, we need to add the appropriate power of 2 if the input
-  // was negative.  This is the same as the optimization in
-  // DAGTypeLegalizer::ExpandIntOp_UNIT_TO_FP, and for it to be safe here,
-  // we must be careful to do the computation in x87 extended precision, not
-  // in SSE. (The generic code can't know it's OK to do this, or how to.)
+  // was negative. We must be careful to do the computation in x87 extended
+  // precision, not in SSE.
   SDVTList Tys = DAG.getVTList(MVT::f80, MVT::Other);
   SDValue Ops[] = { Store, StackSlot };
   SDValue Fild =


        


More information about the llvm-commits mailing list