[Mlir-commits] [mlir] [mlir] test-lower-to-nvvm remove early conversions (PR #74476)

[Guray Ozen]

https://github.com/grypp created https://github.com/llvm/llvm-project/pull/74476

This PR removes vector and memref to llvm conversions. They are run pretty early in the pipeline, it causes problems with address space.

>From 099ddfb108d85e4e4648abc6cf48529e29135e92 Mon Sep 17 00:00:00 2001
From: Guray Ozen <guray.ozen at gmail.com>
Date: Tue, 5 Dec 2023 15:18:38 +0100
Subject: [PATCH] [mlir] test-lower-to-nvvm remove early conversions

This PR removes vector and memref to llvm conversions. They are run pretty early in the pipeline, it causes problems with address space.
---
 mlir/test/lib/Dialect/GPU/TestLowerToNVVM.cpp | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/mlir/test/lib/Dialect/GPU/TestLowerToNVVM.cpp b/mlir/test/lib/Dialect/GPU/TestLowerToNVVM.cpp
index 28f76bde0820a..cad7cfe69d4eb 100644
--- a/mlir/test/lib/Dialect/GPU/TestLowerToNVVM.cpp
+++ b/mlir/test/lib/Dialect/GPU/TestLowerToNVVM.cpp
@@ -23,6 +23,7 @@
 #include "mlir/Conversion/NVVMToLLVM/NVVMToLLVM.h"
 #include "mlir/Conversion/ReconcileUnrealizedCasts/ReconcileUnrealizedCasts.h"
 #include "mlir/Conversion/SCFToControlFlow/SCFToControlFlow.h"
+#include "mlir/Conversion/VectorToGPU/VectorToGPU.h"
 #include "mlir/Conversion/VectorToLLVM/ConvertVectorToLLVMPass.h"
 #include "mlir/Conversion/VectorToSCF/VectorToSCF.h"
 #include "mlir/Dialect/Func/IR/FuncOps.h"
@@ -90,9 +91,7 @@ void buildCommonPassPipeline(OpPassManager &pm,
   pm.addPass(createConvertVectorToSCFPass());
   pm.addPass(createConvertSCFToCFPass());
   pm.addPass(createConvertNVVMToLLVMPass());
-  pm.addPass(createConvertVectorToLLVMPass());
   pm.addPass(createConvertMathToLLVMPass());
-  pm.addPass(createFinalizeMemRefToLLVMConversionPass());
   pm.addPass(createConvertFuncToLLVMPass());
   pm.addPass(memref::createExpandStridedMetadataPass());