Commit 9792f1d8 authored by Amy Zhuang's avatar Amy Zhuang Committed by Scott Cyphers

[MLIR]Add clang flag and skip bf16 tests for MLIR. (#4049)

* [MLIR]Add clang flag and skip bf16 tests for MLIR.

* Address PR feedback.
parent 8cf2bc6f
...@@ -91,6 +91,7 @@ ExternalProject_Add( ...@@ -91,6 +91,7 @@ ExternalProject_Add(
-DLIBOMP_OMPT_SUPPORT=OFF -DLIBOMP_OMPT_SUPPORT=OFF
-DCLANG_BUILD_TOOLS=OFF -DCLANG_BUILD_TOOLS=OFF
-DCLANG_ENABLE_ARCMT=OFF -DCLANG_ENABLE_ARCMT=OFF
-DCLANG_ANALYZER_ENABLE_Z3_SOLVER=OFF
-DCLANG_ENABLE_STATIC_ANALYZER=OFF -DCLANG_ENABLE_STATIC_ANALYZER=OFF
-DLLVM_EXTERNAL_CLANG_SOURCE_DIR=${CLANG_SOURCE_DIR} -DLLVM_EXTERNAL_CLANG_SOURCE_DIR=${CLANG_SOURCE_DIR}
${PASS_LLVM_OPENMP_DIR} ${PASS_LLVM_OPENMP_DIR}
......
...@@ -2155,12 +2155,13 @@ TEST(cpu_test, tensor_copy_from_different_layout) ...@@ -2155,12 +2155,13 @@ TEST(cpu_test, tensor_copy_from_different_layout)
EXPECT_EQ((vector<uint8_t>{1, 4, 2, 5, 3, 6}), read_vector<uint8_t>(b)); EXPECT_EQ((vector<uint8_t>{1, 4, 2, 5, 3, 6}), read_vector<uint8_t>(b));
} }
#ifndef NGRAPH_MLIR_ENABLE
TEST(cpu_test, max_pool_bf16) TEST(cpu_test, max_pool_bf16)
{ {
if (!runtime::cpu::mkldnn_utils::is_bf16_supported()) if (!runtime::cpu::mkldnn_utils::is_bf16_supported())
{ {
// TODO change to skip when there is a new release of gtest // TODO change to skip when there is a new release of gtest
NGRAPH_WARN << "This test is skipped for platform without bf16 support."; NGRAPH_WARN << "This test is skipped for platform without bf16 support and for mlir.";
return; return;
} }
...@@ -2195,7 +2196,7 @@ TEST(cpu_test, convolution_simple_bf16) ...@@ -2195,7 +2196,7 @@ TEST(cpu_test, convolution_simple_bf16)
if (!runtime::cpu::mkldnn_utils::is_bf16_supported()) if (!runtime::cpu::mkldnn_utils::is_bf16_supported())
{ {
// TODO change to skip when there is a new release of gtest // TODO change to skip when there is a new release of gtest
NGRAPH_WARN << "This test is skipped for platform without bf16 support."; NGRAPH_WARN << "This test is skipped for platform without bf16 support and for mlir.";
return; return;
} }
...@@ -2234,6 +2235,7 @@ TEST(cpu_test, convolution_simple_bf16) ...@@ -2234,6 +2235,7 @@ TEST(cpu_test, convolution_simple_bf16)
EXPECT_EQ((vector<bfloat16>{18.0, 24.0, 30.0, 36.0, 18.0, 24.0, 30.0, 36.0}), EXPECT_EQ((vector<bfloat16>{18.0, 24.0, 30.0, 36.0, 18.0, 24.0, 30.0, 36.0}),
read_vector<bfloat16>(result)); read_vector<bfloat16>(result));
} }
#endif
// This tests a backend's implementation of the three parameter version of create_tensor // This tests a backend's implementation of the three parameter version of create_tensor
// Testing using this tensor as a Function input // Testing using this tensor as a Function input
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment