[llvm] r261623 - [ARM] fix initialization of PredictableSelectIsExpensive

Junmo Park via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 23 01:56:58 PST 2016


Author: flyingforyou
Date: Tue Feb 23 03:56:58 2016
New Revision: 261623

URL: http://llvm.org/viewvc/llvm-project?rev=261623&view=rev
Log:
[ARM] fix initialization of PredictableSelectIsExpensive

Summary:
If we want classify OoO or not, using getSchedModel().isOutOfOrder()
could be more proper way than using Subtarget->isLikeA9().

Reviewers: jmolloy, rengolin

Differential Revision: http://reviews.llvm.org/D17433

Modified:
    llvm/trunk/lib/Target/ARM/ARMISelLowering.cpp

Modified: llvm/trunk/lib/Target/ARM/ARMISelLowering.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/ARMISelLowering.cpp?rev=261623&r1=261622&r2=261623&view=diff
==============================================================================
--- llvm/trunk/lib/Target/ARM/ARMISelLowering.cpp (original)
+++ llvm/trunk/lib/Target/ARM/ARMISelLowering.cpp Tue Feb 23 03:56:58 2016
@@ -1042,7 +1042,7 @@ ARMTargetLowering::ARMTargetLowering(con
   setMinStackArgumentAlignment(4);
 
   // Prefer likely predicted branches to selects on out-of-order cores.
-  PredictableSelectIsExpensive = Subtarget->isLikeA9();
+  PredictableSelectIsExpensive = Subtarget->getSchedModel().isOutOfOrder();
 
   setMinFunctionAlignment(Subtarget->isThumb() ? 1 : 2);
 }




More information about the llvm-commits mailing list