[Mlir-commits] [mlir] 3dde260 - [mlir] #include CRunnerUtils.h instead of RunnerUtils.h in SPIRV-runner

llvmlistbot at llvm.org llvmlistbot at llvm.org
Wed Jun 21 01:11:49 PDT 2023


Author: Christian Sigg
Date: 2023-06-21T10:11:46+02:00
New Revision: 3dde2604be2a7de4266c3d8ebe5efd3730b6ccce

URL: https://github.com/llvm/llvm-project/commit/3dde2604be2a7de4266c3d8ebe5efd3730b6ccce
DIFF: https://github.com/llvm/llvm-project/commit/3dde2604be2a7de4266c3d8ebe5efd3730b6ccce.diff

LOG: [mlir] #include CRunnerUtils.h instead of RunnerUtils.h in SPIRV-runner

This avoids bazel builds failing after commit bba2b656110209a3d9863b92c060082479b06ab1 because libmlir_test_spirv_cpu_runner_c_wrappers.so registers the same runner functions twice.

More precisely, this problem only shows up internally at Google because the bazel build does not have that target.

Either way though, it's better to IWYU.

Added: 
    

Modified: 
    mlir/test/mlir-spirv-cpu-runner/mlir_test_spirv_cpu_runner_c_wrappers.cpp

Removed: 
    


################################################################################
diff  --git a/mlir/test/mlir-spirv-cpu-runner/mlir_test_spirv_cpu_runner_c_wrappers.cpp b/mlir/test/mlir-spirv-cpu-runner/mlir_test_spirv_cpu_runner_c_wrappers.cpp
index 8101d7fbbd22f..93433be2dd66a 100644
--- a/mlir/test/mlir-spirv-cpu-runner/mlir_test_spirv_cpu_runner_c_wrappers.cpp
+++ b/mlir/test/mlir-spirv-cpu-runner/mlir_test_spirv_cpu_runner_c_wrappers.cpp
@@ -10,7 +10,7 @@
 //
 //===----------------------------------------------------------------------===//
 
-#include "mlir/ExecutionEngine/RunnerUtils.h"
+#include "mlir/ExecutionEngine/CRunnerUtils.h"
 
 #ifdef _WIN32
 #define EXPORT __declspec(dllexport)


        


More information about the Mlir-commits mailing list