[PowerPC] Use PredictableSelectIsExpensive to enable select to branch in CGP

[ZhangKang]

Summary

There is a patch D19488: [CodeGenPrepare] use branch weight metadata to decide if a select should be turned into a branch, this patch is for CGP, it will try to convert SELECT to branch based on the BranchProbability, when the BranchProbability more than MinPercentageForPredictableBranch(default value is 99%).
The variable PredictableSelectExpensive can be set whether we will do this SELECT to branch optimization, the default value is false. Now only ARM and X86 has set this variable like this:

For PPC, we should also set the variable PredictableSelectIsExpensive to do the select to if based on BranchProbability in CodeGenPrepare.

[nemanjai]

llvm/lib/Target/PowerPC/PPC.td

305

This should certainly be set for Power8 as well.

llvm/lib/Target/PowerPC/PPCISelLowering.cpp

1279

I think that having the same name PredictableSelectIsExpensive as members of both the TLI and the Subtarget is a bit confusing. At the very least, the comment should be a bit more descriptive. Perhaps:

// Let the subtarget (CPU) decide if a predictable select is more
// expensive than the corresponding branch. This information is used
// in CGP to decide when to convert selects into branches.

llvm/lib/Target/PowerPC/PPCSubtarget.h

144

For the sake of convention, leave this uninitialized here and initialize it in PPCSubtarget::initializeEnvironment() just like all the rest of these.

[ZhangKang]

llvm/lib/Target/PowerPC/PPC.td

305

I haven't do the performance for the Power8, so I only enable it for Power9.

llvm/lib/Target/PowerPC/PPCISelLowering.cpp

1279

Have modified the comments.

[ZhangKang]

Modified the patch to follow Nemanjai's comments.

[ZhangKang]

Modify the test case and add set the FeaturePredictableSelectIsExpensive for power8.

[ZhangKang]

llvm/lib/Target/PowerPC/PPC.td

305

Have set the FeaturePredictableSelectIsExpensive for Power8.

[ZhangKang]

Update the full context.

[nemanjai]

LGTM.

[ZhangKang]

Rebase.

[ZhangKang]

Rebase.

llvm/lib/Target/PowerPC/PPC.td

def FeaturePrefixInstrs : SubtargetFeature<"prefix-instrs", "HasPrefixInstrs",
                                            "Enable prefixed instructions",
                                            [FeatureISA3_0, FeatureP8Vector,
                                             FeatureP9Altivec]>;
 def FeaturePCRelativeMemops :
   SubtargetFeature<"pcrelative-memops", "HasPCRelativeMemops", "true",
                    "Enable PC relative Memory Ops",
                    [FeatureISA3_0]>;
 
+
+def FeaturePredictableSelectIsExpensive :
+  SubtargetFeature<"predictable-select-expensive",
+                   "PredictableSelectIsExpensive",
+                   "true",
+                   "Prefer likely predicted branches over selects">;
+
 // Since new processors generally contain a superset of features of those that
 // came before them, the idea is to make implementations of new processors
 // less error prone and easier to read.
 // Namely:
 //     list<SubtargetFeature> P8InheritableFeatures = ...
 //     list<SubtargetFeature> FutureProcessorAddtionalFeatures =
 //         [ features that Power8 does not support but inheritable ]
 //     list<SubtargetFeature> FutureProcessorSpecificFeatures =
list<SubtargetFeature> P7InheritableFeatures = [DirectivePwr7,
                                                   DeprecatedDST,
                                                   FeatureTwoConstNR,
                                                   FeatureUnalignedFloats];
   list<SubtargetFeature> P7SpecificFeatures = [];
   list<SubtargetFeature> P7Features =
     !listconcat(P7InheritableFeatures, P7SpecificFeatures);
 
   // Power8
-  list<SubtargetFeature> P8AdditionalFeatures = [DirectivePwr8,
-                                                 FeatureP8Altivec,
-                                                 FeatureP8Vector,
-                                                 FeatureP8Crypto,
-                                                 FeatureHTM,
-                                                 FeatureDirectMove,
-                                                 FeatureICBT,
-                                                 FeaturePartwordAtomic];
+  list<SubtargetFeature> P8AdditionalFeatures =
+    [DirectivePwr8,
+     FeaturePartwordAtomic,
+     FeaturePredictableSelectIsExpensive
+    ];
+
   list<SubtargetFeature> P8SpecificFeatures = [];
   list<SubtargetFeature> P8InheritableFeatures =
     !listconcat(P7InheritableFeatures, P8AdditionalFeatures);
   list<SubtargetFeature> P8Features =
     !listconcat(P8InheritableFeatures, P8SpecificFeatures);
 
   // Power9
-  list<SubtargetFeature> P9AdditionalFeatures = [DirectivePwr9,
-                                                 FeatureP9Altivec,
-                                                 FeatureP9Vector,
-                                                 FeatureISA3_0];
+  list<SubtargetFeature> P9AdditionalFeatures =
+    [DirectivePwr9,
+     FeatureISA3_0,
+     FeaturePredictableSelectIsExpensive
+    ];
+
   // Some features are unique to Power9 and there is no reason to assume
   // they will be part of any future CPUs. One example is the narrower
   // dispatch for vector operations than scalar ones. For the time being,
   // this list also includes scheduling-related features since we do not have
   // enough info to create custom scheduling strategies for future CPUs.
   list<SubtargetFeature> P9SpecificFeatures = [FeatureVectorsUseTwoUnits,
                                                FeaturePPCPreRASched,
                                                FeaturePPCPostRASched];

llvm/lib/Target/PowerPC/PPCISelLowering.cpp

if (Subtarget.getCPUDirective() == PPC::DIR_E500mc ||
     MaxStoresPerMemset = 128;
     MaxStoresPerMemcpy = 128;
     MaxStoresPerMemmove = 128;
     MaxLoadsPerMemcmp = 128;
   } else {
     MaxLoadsPerMemcmp = 8;
     MaxLoadsPerMemcmpOptSize = 4;
   }
+
+  // Let the subtarget (CPU) decide if a predictable select is more expensive
+  // than the corresponding branch. This information is used in CGP to decide
+  // when to convert selects into branches.
+  PredictableSelectIsExpensive = Subtarget.isPredictableSelectIsExpensive();
 }
 
 /// getMaxByValAlign - Helper for getByValTypeAlignment to determine
 /// the desired ByVal argument alignment.
 static void getMaxByValAlign(Type *Ty, unsigned &MaxAlign,
                              unsigned MaxMaxAlign) {
   if (MaxAlign == MaxMaxAlign)
     return;

llvm/lib/Target/PowerPC/PPCSubtarget.h

protected:
   bool HasHTM;
   bool HasFloat128;
   bool IsISA3_0;
   bool UseLongCalls;
   bool SecurePlt;
   bool VectorsUseTwoUnits;
   bool UsePPCPreRASchedStrategy;
   bool UsePPCPostRASchedStrategy;
+  bool PredictableSelectIsExpensive;
 
   POPCNTDKind HasPOPCNTD;
 
   /// When targeting QPX running a stock PPC64 Linux kernel where the stack
   /// alignment has not been changed, we need to keep the 16-byte alignment
   /// of the stack.
   bool IsQPXStackUnaligned;
 
MCRegister getTOCPointerRegister() const {
     return IsPPC64 ? PPC::X2 : PPC::R2;
   }
 
   MCRegister getStackPointerRegister() const {
     return IsPPC64 ? PPC::X1 : PPC::R1;
   }
 
   bool isXRaySupported() const override { return IsPPC64 && IsLittleEndian; }
+
+  bool isPredictableSelectIsExpensive() const {
+    return PredictableSelectIsExpensive;
+  }
 };
 } // End llvm namespace
 
 #endif

llvm/lib/Target/PowerPC/PPCSubtarget.cpp

void PPCSubtarget::initializeEnvironment() {
   HasHTM = false;
   HasFloat128 = false;
   IsISA3_0 = false;
   UseLongCalls = false;
   SecurePlt = false;
   VectorsUseTwoUnits = false;
   UsePPCPreRASchedStrategy = false;
   UsePPCPostRASchedStrategy = false;
+  PredictableSelectIsExpensive = false;
 
   HasPOPCNTD = POPCNTD_Unavailable;
 }
 
 void PPCSubtarget::initSubtargetFeatures(StringRef CPU, StringRef FS) {
   // Determine default and user specified characteristics
   std::string CPUName = std::string(CPU);
   if (CPUName.empty() || CPU == "generic") {

llvm/test/CodeGen/PowerPC/select-to-branch.mir

+# RUN: llc -verify-machineinstrs -mcpu=pwr9 -mtriple powerpc64le-unknown-linux-gnu \
+# RUN:   -run-pass=codegenprepare -o - %s | FileCheck %s
+# RUN: llc -verify-machineinstrs -mcpu=pwr8 -mtriple powerpc64le-unknown-linux-gnu \
+# RUN:   -run-pass=codegenprepare -o - %s | FileCheck %s
+--- |
+  define i32 @weighted_select1(i32 %a, i32 %b) {
+    %cmp = icmp ne i32 %a, 0
+    %sel = select i1 %cmp, i32 %a, i32 %b, !prof !14
+    ret i32 %sel
+
+  ; If branch_weights > 99% or branch_weights < 1%, the select will be
+  ; converted to branch, here !14 = 99/100, !14 = 99%, so it will do nothing.
+  ; CHECK-LABEL: weighted_select1
+  ; CHECK:         %cmp = icmp ne i32 %a, 0
+  ; CHECK-NEXT:    %sel = select i1 %cmp, i32 %a, i32 %b, !prof !14
+  ; CHECK-NEXT:    ret i32 %sel
+  }
+
+  define i32 @weighted_select2(i32 %a, i32 %b) {
+    %cmp = icmp ne i32 %a, 0
+    %sel = select i1 %cmp, i32 %a, i32 %b, !prof !15
+    ret i32 %sel
+
+  ; If branch_weights > 99% or branch_weights < 1%, the select will be converted
+  ; to branch, here !15 = 100/101, !15 > 99%, so it will convert select to
+  ; branch.
+  ; CHECK-LABEL: weighted_select2
+  ; CHECK:         %cmp = icmp ne i32 %a, 0
+  ; CHECK-NEXT:    br i1 %cmp, label %select.end, label %select.false, !prof !15
+  ; CHECK:       select.false:
+  ; CHECK-NEXT:    br label %select.end
+  ; CHECK:       select.end:
+  ; CHECK-NEXT:    %sel = phi i32 [ %a, %0 ], [ %b, %select.false ]
+  ; CHECK-NEXT:    ret i32 %sel
+  }
+
+  define i32 @weighted_select3(i32 %a, i32 %b) {
+    %cmp = icmp ne i32 %a, 0
+    %sel = select i1 %cmp, i32 %a, i32 %b, !prof !16
+    ret i32 %sel
+
+  ; If branch_weights > 99% or branch_weights < 1%, the select will be converted
+  ; to branch, here !16 = 1/101, !16 < 1%, so it will convert select to branch.
+  ; CHECK-LABEL: weighted_select3
+  ; CHECK:         %cmp = icmp ne i32 %a, 0
+  ; CHECK-NEXT:    br i1 %cmp, label %select.end, label %select.false, !prof !16
+  ; CHECK:       select.false:
+  ; CHECK-NEXT:    br label %select.end
+  ; CHECK:       select.end:
+  ; CHECK-NEXT:    %sel = phi i32 [ %a, %0 ], [ %b, %select.false ]
+  ; CHECK-NEXT:    ret i32 %sel
+  }
+
+  define i32 @unweighted_select(i32 %a, i32 %b) {
+    %cmp = icmp ne i32 %a, 0
+    %sel = select i1 %cmp, i32 %a, i32 %b, !prof !17
+    ret i32 %sel
+
+  ; There is no weight_branch information, so it will do nothing.
+  ; CHECK-LABEL: unweighted_select
+  ; CHECK:         %cmp = icmp ne i32 %a, 0
+  ; CHECK-NEXT:    %sel = select i1 %cmp, i32 %a, i32 %b, !prof !17
+  ; CHECK-NEXT:    ret i32 %sel
+  }
+
+  ; Function Attrs: optsize
+  define i32 @weighted_select_optsize(i32 %a, i32 %b) #0 {
+    %cmp = icmp ne i32 %a, 0
+    %sel = select i1 %cmp, i32 %a, i32 %b, !prof !15
+    ret i32 %sel
+
+  ; This function has used the optsize flag, so it will do nothing.
+  ; CHECK-LABEL: weighted_select_optsize
+  ; CHECK:         %cmp = icmp ne i32 %a, 0
+  ; CHECK-NEXT:    %sel = select i1 %cmp, i32 %a, i32 %b, !prof !15
+  ; CHECK-NEXT:    ret i32 %sel
+  }
+
+  define i32 @weighted_select_pgso(i32 %a, i32 %b) !prof !18 {
+    %cmp = icmp ne i32 %a, 0
+    %sel = select i1 %cmp, i32 %a, i32 %b, !prof !15
+    ret i32 %sel
+
+  ; The function_entry_count of this function is 0, so it will do nothing.
+  ; CHECK-LABEL: weighted_select_pgso
+  ; CHECK:         %cmp = icmp ne i32 %a, 0
+  ; CHECK-NEXT:    %sel = select i1 %cmp, i32 %a, i32 %b, !prof !15
+  ; CHECK-NEXT:    ret i32 %sel
+  }
+  
+  attributes #0 = { optsize }
+  
+  !llvm.module.flags = !{!0}
+  
+  !0 = !{i32 1, !"ProfileSummary", !1}
+  !1 = !{!2, !3, !4, !5, !6, !7, !8, !9}
+  !2 = !{!"ProfileFormat", !"InstrProf"}
+  !3 = !{!"TotalCount", i64 10000}
+  !4 = !{!"MaxCount", i64 10}
+  !5 = !{!"MaxInternalCount", i64 1}
+  !6 = !{!"MaxFunctionCount", i64 1000}
+  !7 = !{!"NumCounts", i64 3}
+  !8 = !{!"NumFunctions", i64 3}
+  !9 = !{!"DetailedSummary", !10}
+  !10 = !{!11, !12, !13}
+  !11 = !{i32 10000, i64 100, i32 1}
+  !12 = !{i32 999000, i64 100, i32 1}
+  !13 = !{i32 999999, i64 1, i32 2}
+  !14 = !{!"branch_weights", i32 1, i32 99}
+  !15 = !{!"branch_weights", i32 1, i32 100}
+  !16 = !{!"branch_weights", i32 100, i32 1}
+  !17 = !{!"branch_weights", i32 0, i32 0}
+  !18 = !{!"function_entry_count", i64 0}
+
+...