[llvm] r352609 - [X86] Remove unnecessary code from the top of handleCompareFP in X86FloatingPoint.cpp.

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 30 00:04:06 PST 2019


Author: ctopper
Date: Wed Jan 30 00:04:06 2019
New Revision: 352609

URL: http://llvm.org/viewvc/llvm-project?rev=352609&view=rev
Log:
[X86] Remove unnecessary code from the top of handleCompareFP in X86FloatingPoint.cpp.

There were checks to ensure some tables were sorted, but those tables aren't used by this function. The same tables are checked in the function that does use them. Maybe this was copy/pasted?

Modified:
    llvm/trunk/lib/Target/X86/X86FloatingPoint.cpp

Modified: llvm/trunk/lib/Target/X86/X86FloatingPoint.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86FloatingPoint.cpp?rev=352609&r1=352608&r2=352609&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86FloatingPoint.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86FloatingPoint.cpp Wed Jan 30 00:04:06 2019
@@ -1368,8 +1368,6 @@ void FPS::handleTwoArgFP(MachineBasicBlo
 /// register arguments and no explicit destinations.
 ///
 void FPS::handleCompareFP(MachineBasicBlock::iterator &I) {
-  ASSERT_SORTED(ForwardST0Table); ASSERT_SORTED(ReverseST0Table);
-  ASSERT_SORTED(ForwardSTiTable); ASSERT_SORTED(ReverseSTiTable);
   MachineInstr &MI = *I;
 
   unsigned NumOperands = MI.getDesc().getNumOperands();




More information about the llvm-commits mailing list