[PATCH] D17433: [ARM] fix initialization of PredictableSelectIsExpensive; NFC

[Junmo Park via llvm-commits]

flyingforyou created this revision.
flyingforyou added a reviewer: jmolloy.
flyingforyou added subscribers: evandro, llvm-commits.
Herald added subscribers: rengolin, aemerson.

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

http://reviews.llvm.org/D17433

Files:
  lib/Target/ARM/ARMISelLowering.cpp

Index: lib/Target/ARM/ARMISelLowering.cpp
===================================================================
--- lib/Target/ARM/ARMISelLowering.cpp
+++ lib/Target/ARM/ARMISelLowering.cpp
@@ -1042,7 +1042,7 @@
   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);
 }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D17433.48446.patch
Type: text/x-patch
Size: 518 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160219/4a8cc216/attachment.bin>