From 22735a0c6ef3eae32415a90e237e6805790a7800 Mon Sep 17 00:00:00 2001 From: Jason Henline Date: Mon, 19 Dec 2016 21:34:07 +0000 Subject: [Axccel] Remove -Wno-missing-braces in build Summary: I originally added the -Wno-missing-braces flag because I thought it was erroneously flagging std::array initializations. Now I realize the extra braces really are desired for these initializations, so I'm turning the warning flag back on. Reviewers: jlebar Subscribers: mgorny, parallel_libs-commits Differential Revision: https://reviews.llvm.org/D27941 --- parallel-libs/acxxel/CMakeLists.txt | 2 +- parallel-libs/acxxel/examples/opencl_example.cpp | 6 +++--- parallel-libs/acxxel/tests/span_test.cpp | 4 ++-- 3 files changed, 6 insertions(+), 6 deletions(-) (limited to 'parallel-libs') diff --git a/parallel-libs/acxxel/CMakeLists.txt b/parallel-libs/acxxel/CMakeLists.txt index 4b21da32e89..547dd62d2fb 100644 --- a/parallel-libs/acxxel/CMakeLists.txt +++ b/parallel-libs/acxxel/CMakeLists.txt @@ -35,7 +35,7 @@ set(CMAKE_CXX_STANDARD 11) set(CMAKE_CXX_STANDARD_REQUIRED ON) # Add warning flags. -set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wextra -Wno-missing-braces") +set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wextra") add_library( acxxel diff --git a/parallel-libs/acxxel/examples/opencl_example.cpp b/parallel-libs/acxxel/examples/opencl_example.cpp index 713daac2627..4e3cf0795b5 100644 --- a/parallel-libs/acxxel/examples/opencl_example.cpp +++ b/parallel-libs/acxxel/examples/opencl_example.cpp @@ -57,9 +57,9 @@ void saxpy(float A, std::array &X, const std::array &Y) { int main() { float A = 2.f; - std::array X = {0.f, 1.f, 2.f}; - std::array Y = {3.f, 4.f, 5.f}; - std::array Expected = {3.f, 6.f, 9.f}; + std::array X{{0.f, 1.f, 2.f}}; + std::array Y{{3.f, 4.f, 5.f}}; + std::array Expected{{3.f, 6.f, 9.f}}; saxpy(A, X, Y); for (int I = 0; I < 3; ++I) if (X[I] != Expected[I]) { diff --git a/parallel-libs/acxxel/tests/span_test.cpp b/parallel-libs/acxxel/tests/span_test.cpp index eee5aefef05..a425ceb4af2 100644 --- a/parallel-libs/acxxel/tests/span_test.cpp +++ b/parallel-libs/acxxel/tests/span_test.cpp @@ -79,12 +79,12 @@ TEST(Span, ArrayConstruction) { } TEST(Span, StdArrayConstruction) { - std::array Array{0, 1, 2}; + std::array Array{{0, 1, 2}}; acxxel::Span Span(Array); EXPECT_EQ(Span.data(), Array.data()); EXPECT_EQ(static_cast(Span.size()), Array.size()); - std::array ConstArray{0, 1, 2}; + std::array ConstArray{{0, 1, 2}}; acxxel::Span ConstSpan(ConstArray); EXPECT_EQ(ConstSpan.data(), ConstArray.data()); EXPECT_EQ(static_cast(ConstSpan.size()), ConstArray.size()); -- cgit v1.2.3