[mlir][test] NFC - Rename cblas to mlir_test_cblas

[nicolasvasilache]

Summary

The "cblas" lib under mlir/test is meant as a simple integration demonstration.
However it is installed and ends up conflicting with external projects who want to
define the real cblas.
Rename to avoid conflicts.

[isuruf]

LGTM. I checked out this branch using arc and it builds fine for me and the issue is fixed. Thanks for fixing this issue.

[nicolasvasilache]

Update

mlir/test/CMakeLists.txt

set(MLIR_TEST_DEPENDS
   FileCheck count not
   MLIRUnitTests
   mlir-cpu-runner
   mlir-edsc-builder-api-test
   mlir-opt
   mlir-sdbm-api-test
   mlir-tblgen
   mlir-translate
-  cblas
-  cblas_interface
+  mlir_test_cblas
+  mlir_test_cblas_interface
   mlir_runner_utils
   mlir_c_runner_utils
   )
 
 if(LLVM_BUILD_EXAMPLES)
   list(APPEND MLIR_TEST_DEPENDS
     toyc-ch1
     toyc-ch2

mlir/test/mlir-cpu-runner/CMakeLists.txt

 set(LLVM_OPTIONAL_SOURCES
-  cblas.cpp
-  cblas_interface.cpp
+  mlir_test_cblas.cpp
+  mlir_test_cblas_interface.cpp
   )
 
-add_llvm_library(cblas SHARED cblas.cpp)
-target_compile_definitions(cblas PRIVATE cblas_EXPORTS)
+add_llvm_library(mlir_test_cblas SHARED mlir_test_cblas.cpp)
+target_compile_definitions(mlir_test_cblas PRIVATE mlir_test_cblas_EXPORTS)
 
-add_llvm_library(cblas_interface SHARED cblas_interface.cpp)
-target_link_libraries(cblas_interface PRIVATE cblas)
-target_compile_definitions(cblas_interface PRIVATE cblas_interface_EXPORTS)
+add_llvm_library(mlir_test_cblas_interface SHARED mlir_test_cblas_interface.cpp)
+target_link_libraries(mlir_test_cblas_interface PRIVATE mlir_test_cblas)
+target_compile_definitions(mlir_test_cblas_interface PRIVATE mlir_test_cblas_interface_EXPORTS)
 

mlir/test/mlir-cpu-runner/include/cblas.h

-//===- cblas.h - Simple Blas subset ---------------------------------------===//
-//
-// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
-// See https://llvm.org/LICENSE.txt for license information.
-// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
-//
-//===----------------------------------------------------------------------===//
-#ifndef MLIR_CPU_RUNNER_CBLAS_H_
-#define MLIR_CPU_RUNNER_CBLAS_H_
-
-#include "mlir/ExecutionEngine/RunnerUtils.h"
-
-#ifdef _WIN32
-#ifndef MLIR_CBLAS_EXPORT
-#ifdef cblas_EXPORTS
-/* We are building this library */
-#define MLIR_CBLAS_EXPORT __declspec(dllexport)
-#else
-/* We are using this library */
-#define MLIR_CBLAS_EXPORT __declspec(dllimport)
-#endif // cblas_EXPORTS
-#endif // MLIR_CBLAS_EXPORT
-#else
-#define MLIR_CBLAS_EXPORT
-#endif // _WIN32
-
-/// This reproduces a minimal subset of cblas to allow integration testing
-/// without explicitly requiring a dependence on an external library.
-/// Without loss of generality, various cblas implementations may be swapped in
-/// by including the proper headers and linking with the proper library.
-enum CBLAS_ORDER { CblasRowMajor = 101, CblasColMajor = 102 };
-enum CBLAS_TRANSPOSE {
-  CblasNoTrans = 111,
-  CblasTrans = 112,
-  CblasConjTrans = 113
-};
-
-extern "C" MLIR_CBLAS_EXPORT float cblas_sdot(const int N, const float *X,
-                                              const int incX, const float *Y,
-                                              const int incY);
-
-extern "C" MLIR_CBLAS_EXPORT void
-cblas_sgemm(const enum CBLAS_ORDER Order, const enum CBLAS_TRANSPOSE TransA,
-            const enum CBLAS_TRANSPOSE TransB, const int M, const int N,
-            const int K, const float alpha, const float *A, const int lda,
-            const float *B, const int ldb, const float beta, float *C,
-            const int ldc);
-
-#endif // MLIR_CPU_RUNNER_CBLAS_H_

mlir/test/mlir-cpu-runner/include/mlir_test_cblas.h

+//===- mlir_test_cblas.h - Simple Blas subset -----------------------------===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+#ifndef MLIR_CPU_RUNNER_MLIR_TEST_CBLAS_H_
+#define MLIR_CPU_RUNNER_MLIR_TEST_CBLAS_H_
+
+#include "mlir/ExecutionEngine/RunnerUtils.h"
+
+#ifdef _WIN32
+#ifndef MLIR_TEST_CBLAS_EXPORT
+#ifdef mlir_test_cblas_EXPORTS
+/* We are building this library */
+#define MLIR_TEST_CBLAS_EXPORT __declspec(dllexport)
+#else
+/* We are using this library */
+#define MLIR_TEST_CBLAS_EXPORT __declspec(dllimport)
+#endif // mlir_test_cblas_EXPORTS
+#endif // MLIR_TEST_CBLAS_EXPORT
+#else
+#define MLIR_TEST_CBLAS_EXPORT
+#endif // _WIN32
+
+/// This reproduces a minimal subset of mlir_test_cblas to allow integration
+/// testing without explicitly requiring a dependence on an external library.
+/// Without loss of generality, various mlir_test_cblas implementations may be
+/// swapped in by including the proper headers and linking with the proper
+/// library.
+enum CBLAS_ORDER { CblasRowMajor = 101, CblasColMajor = 102 };
+enum CBLAS_TRANSPOSE {
+  CblasNoTrans = 111,
+  CblasTrans = 112,
+  CblasConjTrans = 113
+};
+
+extern "C" MLIR_TEST_CBLAS_EXPORT float
+mlir_test_cblas_sdot(const int N, const float *X, const int incX,
+                     const float *Y, const int incY);
+
+extern "C" MLIR_TEST_CBLAS_EXPORT void mlir_test_cblas_sgemm(
+    const enum CBLAS_ORDER Order, const enum CBLAS_TRANSPOSE TransA,
+    const enum CBLAS_TRANSPOSE TransB, const int M, const int N, const int K,
+    const float alpha, const float *A, const int lda, const float *B,
+    const int ldb, const float beta, float *C, const int ldc);
+
+#endif // MLIR_CPU_RUNNER_MLIR_TEST_CBLAS_H_

mlir/test/mlir-cpu-runner/include/mlir_test_cblas_interface.h

-//===- cblas_interface.h - Simple Blas subset interface -------------------===//
+//===- mlir_test_cblas_interface.h - Simple Blas subset interface ---------===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 //
 //===----------------------------------------------------------------------===//
-#ifndef MLIR_CPU_RUNNER_CBLAS_INTERFACE_H_
-#define MLIR_CPU_RUNNER_CBLAS_INTERFACE_H_
+#ifndef MLIR_CPU_RUNNER_MLIR_TEST_CBLAS_INTERFACE_H_
+#define MLIR_CPU_RUNNER_MLIR_TEST_CBLAS_INTERFACE_H_
 
 #include "mlir/ExecutionEngine/RunnerUtils.h"
 
 #ifdef _WIN32
-#ifndef MLIR_CBLAS_INTERFACE_EXPORT
-#ifdef cblas_interface_EXPORTS
+#ifndef MLIR_TEST_CBLAS_INTERFACE_EXPORT
+#ifdef mlir_test_cblas_interface_EXPORTS
 /* We are building this library */
-#define MLIR_CBLAS_INTERFACE_EXPORT __declspec(dllexport)
+#define MLIR_TEST_CBLAS_INTERFACE_EXPORT __declspec(dllexport)
 #else
 /* We are using this library */
-#define MLIR_CBLAS_INTERFACE_EXPORT __declspec(dllimport)
-#endif // cblas_interface_EXPORTS
-#endif // MLIR_CBLAS_INTERFACE_EXPORT
+#define MLIR_TEST_CBLAS_INTERFACE_EXPORT __declspec(dllimport)
+#endif // mlir_test_cblas_interface_EXPORTS
+#endif // MLIR_TEST_CBLAS_INTERFACE_EXPORT
 #else
-#define MLIR_CBLAS_INTERFACE_EXPORT
+#define MLIR_TEST_CBLAS_INTERFACE_EXPORT
 #endif // _WIN32
 
-extern "C" MLIR_CBLAS_INTERFACE_EXPORT void
+extern "C" MLIR_TEST_CBLAS_INTERFACE_EXPORT void
 _mlir_ciface_linalg_fill_viewf32_f32(StridedMemRefType<float, 0> *X, float f);
 
-extern "C" MLIR_CBLAS_INTERFACE_EXPORT void
+extern "C" MLIR_TEST_CBLAS_INTERFACE_EXPORT void
 _mlir_ciface_linalg_fill_viewsxf32_f32(StridedMemRefType<float, 1> *X, float f);
 
-extern "C" MLIR_CBLAS_INTERFACE_EXPORT void
+extern "C" MLIR_TEST_CBLAS_INTERFACE_EXPORT void
 _mlir_ciface_linalg_fill_viewsxsxf32_f32(StridedMemRefType<float, 2> *X,
                                          float f);
 
-extern "C" MLIR_CBLAS_INTERFACE_EXPORT void
+extern "C" MLIR_TEST_CBLAS_INTERFACE_EXPORT void
 _mlir_ciface_linalg_copy_viewf32_viewf32(StridedMemRefType<float, 0> *I,
                                          StridedMemRefType<float, 0> *O);
 
-extern "C" MLIR_CBLAS_INTERFACE_EXPORT void
+extern "C" MLIR_TEST_CBLAS_INTERFACE_EXPORT void
 _mlir_ciface_linalg_copy_viewsxf32_viewsxf32(StridedMemRefType<float, 1> *I,
                                              StridedMemRefType<float, 1> *O);
 
-extern "C" MLIR_CBLAS_INTERFACE_EXPORT void
+extern "C" MLIR_TEST_CBLAS_INTERFACE_EXPORT void
 _mlir_ciface_linalg_copy_viewsxsxf32_viewsxsxf32(
     StridedMemRefType<float, 2> *I, StridedMemRefType<float, 2> *O);
 
-extern "C" MLIR_CBLAS_INTERFACE_EXPORT void
+extern "C" MLIR_TEST_CBLAS_INTERFACE_EXPORT void
 _mlir_ciface_linalg_dot_viewsxf32_viewsxf32_viewf32(
     StridedMemRefType<float, 1> *X, StridedMemRefType<float, 1> *Y,
     StridedMemRefType<float, 0> *Z);
 
-extern "C" MLIR_CBLAS_INTERFACE_EXPORT void
+extern "C" MLIR_TEST_CBLAS_INTERFACE_EXPORT void
 _mlir_ciface_linalg_matmul_viewsxsxf32_viewsxsxf32_viewsxsxf32(
     StridedMemRefType<float, 2> *A, StridedMemRefType<float, 2> *B,
     StridedMemRefType<float, 2> *C);
 
-#endif // MLIR_CPU_RUNNER_CBLAS_INTERFACE_H_
+#endif // MLIR_CPU_RUNNER_MLIR_TEST_CBLAS_INTERFACE_H_

mlir/test/mlir-cpu-runner/linalg_integration_test.mlir

-// RUN: mlir-opt %s -convert-linalg-to-llvm | mlir-cpu-runner -e dot -entry-point-result=f32 -shared-libs=%linalg_test_lib_dir/libcblas%shlibext,%linalg_test_lib_dir/libcblas_interface%shlibext | FileCheck %s
-// RUN: mlir-opt %s -convert-linalg-to-loops -convert-linalg-to-llvm | mlir-cpu-runner -e dot -entry-point-result=f32 -shared-libs=%linalg_test_lib_dir/libcblas%shlibext,%linalg_test_lib_dir/libcblas_interface%shlibext | FileCheck %s
-// RUN: mlir-opt %s -convert-linalg-to-llvm | mlir-cpu-runner -e matmul -entry-point-result=f32 -shared-libs=%linalg_test_lib_dir/libcblas%shlibext,%linalg_test_lib_dir/libcblas_interface%shlibext | FileCheck %s
-// RUN: mlir-opt %s -convert-linalg-to-loops -convert-linalg-to-llvm | mlir-cpu-runner -e matmul -entry-point-result=f32 -shared-libs=%linalg_test_lib_dir/libcblas%shlibext,%linalg_test_lib_dir/libcblas_interface%shlibext | FileCheck %s
-// RUN: mlir-opt %s -linalg-tile -linalg-tile-sizes=2,3,4 -linalg-promote-subviews -convert-linalg-to-loops -convert-linalg-to-llvm | mlir-cpu-runner -e matmul -entry-point-result=f32 -shared-libs=%linalg_test_lib_dir/libcblas%shlibext,%linalg_test_lib_dir/libcblas_interface%shlibext | FileCheck %s
-// RUN: mlir-opt %s -linalg-tile -linalg-tile-sizes=2,3,4 -linalg-promote-subviews -convert-linalg-to-llvm | mlir-cpu-runner -e matmul -entry-point-result=f32 -shared-libs=%linalg_test_lib_dir/libcblas%shlibext,%linalg_test_lib_dir/libcblas_interface%shlibext | FileCheck %s
+// RUN: mlir-opt %s -convert-linalg-to-llvm | mlir-cpu-runner -e dot -entry-point-result=f32 -shared-libs=%linalg_test_lib_dir/libmlir_test_cblas%shlibext,%linalg_test_lib_dir/libmlir_test_cblas_interface%shlibext | FileCheck %s
+// RUN: mlir-opt %s -convert-linalg-to-loops -convert-linalg-to-llvm | mlir-cpu-runner -e dot -entry-point-result=f32 -shared-libs=%linalg_test_lib_dir/libmlir_test_cblas%shlibext,%linalg_test_lib_dir/libmlir_test_cblas_interface%shlibext | FileCheck %s
+// RUN: mlir-opt %s -convert-linalg-to-llvm | mlir-cpu-runner -e matmul -entry-point-result=f32 -shared-libs=%linalg_test_lib_dir/libmlir_test_cblas%shlibext,%linalg_test_lib_dir/libmlir_test_cblas_interface%shlibext | FileCheck %s
+// RUN: mlir-opt %s -convert-linalg-to-loops -convert-linalg-to-llvm | mlir-cpu-runner -e matmul -entry-point-result=f32 -shared-libs=%linalg_test_lib_dir/libmlir_test_cblas%shlibext,%linalg_test_lib_dir/libmlir_test_cblas_interface%shlibext | FileCheck %s
+// RUN: mlir-opt %s -linalg-tile -linalg-tile-sizes=2,3,4 -linalg-promote-subviews -convert-linalg-to-loops -convert-linalg-to-llvm | mlir-cpu-runner -e matmul -entry-point-result=f32 -shared-libs=%linalg_test_lib_dir/libmlir_test_cblas%shlibext,%linalg_test_lib_dir/libmlir_test_cblas_interface%shlibext | FileCheck %s
+// RUN: mlir-opt %s -linalg-tile -linalg-tile-sizes=2,3,4 -linalg-promote-subviews -convert-linalg-to-llvm | mlir-cpu-runner -e matmul -entry-point-result=f32 -shared-libs=%linalg_test_lib_dir/libmlir_test_cblas%shlibext,%linalg_test_lib_dir/libmlir_test_cblas_interface%shlibext | FileCheck %s
 
 #strided1D = affine_map<(d0) -> (d0)>
 #strided2D = affine_map<(d0, d1)[s0] -> (d0 * s0 + d1)>
 
 // Creates and returns a 1-D buffer of size %s filled with the value %f
 func @alloc_filled_f32(%s : index, %f : f32) -> memref<?xi8> {
   %c0 = constant 0 : index
   %c1 = constant 1 : index

mlir/test/mlir-cpu-runner/mlir_test_cblas.cpp

-//===- cblas.cpp - Simple Blas subset implementation ----------------------===//
+//===- mlir_test_cblas.cpp - Simple Blas subset implementation ------------===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 //
 //===----------------------------------------------------------------------===//
 //
 // Simple Blas subset implementation.
 //
 //===----------------------------------------------------------------------===//
 
-#include "include/cblas.h"
+#include "include/mlir_test_cblas.h"
 #include <assert.h>
 
-extern "C" float cblas_sdot(const int N, const float *X, const int incX,
-                            const float *Y, const int incY) {
+extern "C" float mlir_test_cblas_sdot(const int N, const float *X,
+                                      const int incX, const float *Y,
+                                      const int incY) {
   float res = 0.0f;
   for (int i = 0; i < N; ++i)
     res += X[i * incX] * Y[i * incY];
   return res;
 }
 
-extern "C" void cblas_sgemm(const enum CBLAS_ORDER Order,
-                            const enum CBLAS_TRANSPOSE TransA,
-                            const enum CBLAS_TRANSPOSE TransB, const int M,
-                            const int N, const int K, const float alpha,
-                            const float *A, const int lda, const float *B,
-                            const int ldb, const float beta, float *C,
-                            const int ldc) {
+extern "C" void mlir_test_cblas_sgemm(
+    const enum CBLAS_ORDER Order, const enum CBLAS_TRANSPOSE TransA,
+    const enum CBLAS_TRANSPOSE TransB, const int M, const int N, const int K,
+    const float alpha, const float *A, const int lda, const float *B,
+    const int ldb, const float beta, float *C, const int ldc) {
   assert(Order == CBLAS_ORDER::CblasRowMajor);
   assert(TransA == CBLAS_TRANSPOSE::CblasNoTrans);
   assert(TransB == CBLAS_TRANSPOSE::CblasNoTrans);
   for (int m = 0; m < M; ++m) {
     auto *pA = A + m * lda;
     auto *pC = C + m * ldc;
     for (int n = 0; n < N; ++n) {
       float c = pC[n];

mlir/test/mlir-cpu-runner/mlir_test_cblas_interface.cpp

-//===- cblas_interface.cpp - Simple Blas subset interface -----------------===//
+//===- mlir_test_cblas_interface.cpp - Simple Blas subset interface -------===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 //
 //===----------------------------------------------------------------------===//
 //
 // Simple Blas subset interface implementation.
 //
 //===----------------------------------------------------------------------===//
 
-#include "include/cblas_interface.h"
-#include "include/cblas.h"
+#include "include/mlir_test_cblas_interface.h"
+#include "include/mlir_test_cblas.h"
 #include <assert.h>
 #include <iostream>
 
 extern "C" void
 _mlir_ciface_linalg_fill_viewf32_f32(StridedMemRefType<float, 0> *X, float f) {
   X->data[X->offset] = f;
 }
 
extern "C" void _mlir_ciface_linalg_dot_viewsxf32_viewsxf32_viewf32(
   if (X->strides[0] != 1 || Y->strides[0] != 1 || X->sizes[0] != Y->sizes[0]) {
     std::cerr << "Incompatible strided memrefs\n";
     printMemRefMetaData(std::cerr, *X);
     printMemRefMetaData(std::cerr, *Y);
     printMemRefMetaData(std::cerr, *Z);
     return;
   }
   Z->data[Z->offset] +=
-      cblas_sdot(X->sizes[0], X->data + X->offset, X->strides[0],
+      mlir_test_cblas_sdot(X->sizes[0], X->data + X->offset, X->strides[0],
                            Y->data + Y->offset, Y->strides[0]);
 }
 
 extern "C" void _mlir_ciface_linalg_matmul_viewsxsxf32_viewsxsxf32_viewsxsxf32(
     StridedMemRefType<float, 2> *A, StridedMemRefType<float, 2> *B,
     StridedMemRefType<float, 2> *C) {
   if (A->strides[1] != B->strides[1] || A->strides[1] != C->strides[1] ||
       A->strides[1] != 1 || A->sizes[0] < A->strides[1] ||
       B->sizes[0] < B->strides[1] || C->sizes[0] < C->strides[1] ||
       C->sizes[0] != A->sizes[0] || C->sizes[1] != B->sizes[1] ||
       A->sizes[1] != B->sizes[0]) {
     printMemRefMetaData(std::cerr, *A);
     printMemRefMetaData(std::cerr, *B);
     printMemRefMetaData(std::cerr, *C);
     return;
   }
-  cblas_sgemm(CBLAS_ORDER::CblasRowMajor, CBLAS_TRANSPOSE::CblasNoTrans,
-              CBLAS_TRANSPOSE::CblasNoTrans, C->sizes[0], C->sizes[1],
-              A->sizes[1], 1.0f, A->data + A->offset, A->strides[0],
-              B->data + B->offset, B->strides[0], 1.0f, C->data + C->offset,
-              C->strides[0]);
+  mlir_test_cblas_sgemm(
+      CBLAS_ORDER::CblasRowMajor, CBLAS_TRANSPOSE::CblasNoTrans,
+      CBLAS_TRANSPOSE::CblasNoTrans, C->sizes[0], C->sizes[1], A->sizes[1],
+      1.0f, A->data + A->offset, A->strides[0], B->data + B->offset,
+      B->strides[0], 1.0f, C->data + C->offset, C->strides[0]);
 }

mlir/test/mlir-cpu-runner/unranked_memref.mlir

-// RUN: mlir-opt %s -convert-linalg-to-loops -convert-linalg-to-llvm -convert-std-to-llvm | mlir-cpu-runner -e main -entry-point-result=void -shared-libs=%linalg_test_lib_dir/libmlir_runner_utils%shlibext,%linalg_test_lib_dir/libcblas%shlibext,%linalg_test_lib_dir/libcblas_interface%shlibext | FileCheck %s
+// RUN: mlir-opt %s -convert-linalg-to-loops -convert-linalg-to-llvm -convert-std-to-llvm | mlir-cpu-runner -e main -entry-point-result=void -shared-libs=%linalg_test_lib_dir/libmlir_runner_utils%shlibext,%linalg_test_lib_dir/libmlir_test_cblas%shlibext,%linalg_test_lib_dir/libmlir_test_cblas_interface%shlibext | FileCheck %s
 
 // CHECK: rank = 2
 // CHECK: rank = 2
 // CHECK-SAME: sizes = [10, 3]
 // CHECK-SAME: strides = [3, 1]
 // CHECK-COUNT-10: [10, 10, 10]
 //
 // CHECK: rank = 2