Merge pull request #198 from buddy-compiler/revert-197-fix-legalize-f…

…uncname Revert "Fix gemmini legalize funcname."
buddy-compiler · Sep 8, 2023 · da0c057 · da0c057
2 parents 1e94854 + af14ca6
commit da0c057
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/midend/include/Dialect/Gemmini/Transform.h b/midend/include/Dialect/Gemmini/Transform.h
@@ -50,7 +50,7 @@ void populateGemminiLegalizeForLLVMExportPatterns(LLVMTypeConverter &converter,
                                                   RewritePatternSet &patterns,
                                                   int64_t dim, int64_t addrLen, 
                                                   size_t sizeOfElemT, size_t sizeOfAccT);
-void configureGemminiLegalizeForExportTarget(LLVMConversionTarget &target);
+void configureGemminiegalizeForExportTarget(LLVMConversionTarget &target);
 
 } // namespace mlir
 

diff --git a/midend/lib/Conversion/LowerGemmini/LowerGemminiPass.cpp b/midend/lib/Conversion/LowerGemmini/LowerGemminiPass.cpp
@@ -175,6 +175,7 @@ class LowerGemminiToLLVMPass
 
   // Override explicitly to allow conditional dialect dependence.
   void getDependentDialects(DialectRegistry &registry) const override {
+    registry.insert<LLVM::LLVMDialect>();
     registry.insert<LLVM::LLVMDialect>();
     registry.insert<arith::ArithDialect>();
     registry.insert<memref::MemRefDialect>();
@@ -202,7 +203,7 @@ void LowerGemminiToLLVMPass::runOnOperation() {
   LLVMTypeConverter converter(context);
   RewritePatternSet patterns(context);
   LLVMConversionTarget target(*context);
-  configureGemminiLegalizeForExportTarget(target);
+  configureGemminiegalizeForExportTarget(target);
   populateGemminiLegalizeForLLVMExportPatterns(
       converter, patterns, dim, addrLen, sizeOfElemT, sizeOfAccT);
   populateAffineToStdConversionPatterns(patterns);

diff --git a/midend/lib/Dialect/Gemmini/Transforms/LegalizeForLLVMExport.cpp b/midend/lib/Dialect/Gemmini/Transforms/LegalizeForLLVMExport.cpp
@@ -1380,7 +1380,7 @@ void mlir::populateGemminiLegalizeForLLVMExportPatterns(
                                         sizeOfAccT);
 }
 
-void mlir::configureGemminiLegalizeForExportTarget(
+void mlir::configureGemminiegalizeForExportTarget(
     LLVMConversionTarget &target) {
   target.addLegalOp<
       Flush_IntrOp, ConfigSt_IntrOp, ConifgLd_IntrOp, ConfigEX_IntrOp,