[PATCH] D60177: [PowerPC] initialize SchedModel according to platform

ChenZheng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 2 21:32:23 PDT 2019


shchenz created this revision.
shchenz added reviewers: nemanjai, steven.zhang, jsji, hfinkel, lei.
Herald added subscribers: kbarton, hiraditya.
Herald added a project: LLVM.

This is a typo fix.
We should initialize SchedModel according to platform.


https://reviews.llvm.org/D60177

Files:
  llvm/lib/Target/PowerPC/PPCCTRLoops.cpp
  llvm/test/CodeGen/PowerPC/ctrloop-shortLoops.ll


Index: llvm/test/CodeGen/PowerPC/ctrloop-shortLoops.ll
===================================================================
--- llvm/test/CodeGen/PowerPC/ctrloop-shortLoops.ll
+++ llvm/test/CodeGen/PowerPC/ctrloop-shortLoops.ll
@@ -122,7 +122,7 @@
 ; a2q should use mtctr, but pwr8 should not use mtctr.
 define signext i32 @testTripCount5() {
 ; CHECK-LABEL: testTripCount5:
-; CHECK-PWR8: mtctr
+; CHECK-PWR8-NOT: mtctr
 ; CHECK-A2Q: mtctr
  
 entry:
Index: llvm/lib/Target/PowerPC/PPCCTRLoops.cpp
===================================================================
--- llvm/lib/Target/PowerPC/PPCCTRLoops.cpp
+++ llvm/lib/Target/PowerPC/PPCCTRLoops.cpp
@@ -202,6 +202,7 @@
   auto *TLIP = getAnalysisIfAvailable<TargetLibraryInfoWrapperPass>();
   LibInfo = TLIP ? &TLIP->getTLI() : nullptr;
   PreserveLCSSA = mustPreserveAnalysisID(LCSSAID);
+  SchedModel.init(STI);
 
   bool MadeChange = false;
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D60177.193428.patch
Type: text/x-patch
Size: 903 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190403/fd6dde0c/attachment.bin>


More information about the llvm-commits mailing list