update

yifeizh2 · yifeizh2 · commit a170e7d64cdb · 2024-10-09T01:40:33.000-07:00
diff --git a/lib/gc/Analysis/MatmulConfigAnalysis.cpp b/lib/gc/Analysis/MatmulConfigAnalysis.cpp
@@ -403,7 +403,7 @@ bool readConfigFromAttrs(MatmulConfig &config, ArrayRef<NamedAttribute> attrs,
   if (validateConfig(config, shape)) {
     return cfgItemCnt == 9;
   } else {
-    LLVM_DEBUG(llvm::dbgs() << "The predefined config is invalid\n");
+    assert(validateConfig(config, shape) && "config is invalid");
     return false;
   }
 }
@@ -519,6 +519,8 @@ MatmulConfig MatmulConfigAnalysis::getConfig() {
         }
         if (!configCandidates.empty())
           config = configCandidates[0];
+
+        assert(validateConfig(config, shape) && "config is invalid");
       }
 
       LLVM_DEBUG(llvm::dbgs()

Original file line number	Diff line number	Diff line change
`@@ -403,7 +403,7 @@ bool readConfigFromAttrs(MatmulConfig &config, ArrayRef<NamedAttribute> attrs,`
`403`	`403`	`if (validateConfig(config, shape)) {`
`404`	`404`	`return cfgItemCnt == 9;`
`405`	`405`	`} else {`
`406`		`- LLVM_DEBUG(llvm::dbgs() << "The predefined config is invalid\n");`
	`406`	`+ assert(validateConfig(config, shape) && "config is invalid");`
`407`	`407`	`return false;`
`408`	`408`	`}`
`409`	`409`	`}`
`@@ -519,6 +519,8 @@ MatmulConfig MatmulConfigAnalysis::getConfig() {`
`519`	`519`	`}`
`520`	`520`	`if (!configCandidates.empty())`
`521`	`521`	`config = configCandidates[0];`
	`522`	`+`
	`523`	`+ assert(validateConfig(config, shape) && "config is invalid");`
`522`	`524`	`}`
`523`	`525`
`524`	`526`	`LLVM_DEBUG(llvm::dbgs()`