[PATCH] D15792: [AArch64] Turn off PredictableSelectIsExpensive on the Cortex-A57

[Junmo Park via llvm-commits]

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

There is some significant improvement in commertial benchmark(~10%).
No regression in spec2000, 2006, test-suite.
Test Env: Juno, Cortex-A57 1.1Ghz
Base Rev: r256406

http://reviews.llvm.org/D15792

Files:
  lib/Target/AArch64/AArch64ISelLowering.cpp

Index: lib/Target/AArch64/AArch64ISelLowering.cpp
===================================================================
--- lib/Target/AArch64/AArch64ISelLowering.cpp
+++ lib/Target/AArch64/AArch64ISelLowering.cpp
@@ -622,10 +622,6 @@
       setOperationAction(ISD::FROUND, Ty, Legal);
     }
   }
-
-  // Prefer likely predicted branches to selects on out-of-order cores.
-  if (Subtarget->isCortexA57())
-    PredictableSelectIsExpensive = true;
 }
 
 void AArch64TargetLowering::addTypeForNEON(EVT VT, EVT PromotedBitwiseVT) {


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D15792.43669.patch
Type: text/x-patch
Size: 528 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151228/9256f780/attachment.bin>