[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