Skip to content

Commit 15279e7

Browse files
carlosgalvezpCarlos Gálvez
and
Carlos Gálvez
authored
[OpenMP] Remove -Wno-enum-constexpr-conversion (#81318)
This effectively reverts commit 9ff0cc7. For some reason "git revert" lead to "no changes" after fixing conflicts, so a clean revert was not possible. The original issue (#57022) is no longer reproducible even with this patch, so we can remove the suppression. This is in line with our goal to make -Wenum-constexpr-conversion a non-downgradeable error, see #59036. Co-authored-by: Carlos Gálvez <carlos.galvez@zenseact.com>
1 parent 082439c commit 15279e7

File tree

2 files changed

+0
-2
lines changed

2 files changed

+0
-2
lines changed

openmp/cmake/HandleOpenMPOptions.cmake

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,6 @@ append_if(OPENMP_HAVE_WSIGN_COMPARE_FLAG "-Wsign-compare" CMAKE_C_FLAGS CMAKE_CX
4141
# printed. Therefore, check for whether the compiler supports options in the
4242
# form -W<foo>, and if supported, add the corresponding -Wno-<foo> option.
4343

44-
append_if(OPENMP_HAVE_WENUM_CONSTEXPR_CONVERSION_FLAG "-Wno-enum-constexpr-conversion" CMAKE_C_FLAGS CMAKE_CXX_FLAGS)
4544
append_if(OPENMP_HAVE_WEXTRA_FLAG "-Wno-extra" CMAKE_C_FLAGS CMAKE_CXX_FLAGS)
4645
append_if(OPENMP_HAVE_WPEDANTIC_FLAG "-Wno-pedantic" CMAKE_C_FLAGS CMAKE_CXX_FLAGS)
4746
append_if(OPENMP_HAVE_WMAYBE_UNINITIALIZED_FLAG "-Wno-maybe-uninitialized" CMAKE_C_FLAGS CMAKE_CXX_FLAGS)

openmp/cmake/config-ix.cmake

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,6 @@ check_cxx_compiler_flag(-Wsign-compare OPENMP_HAVE_WSIGN_COMPARE_FLAG)
3333
# printed. Therefore, check for whether the compiler supports options in the
3434
# form -W<foo>, and if supported, add the corresponding -Wno-<foo> option.
3535

36-
check_cxx_compiler_flag(-Wenum-constexpr-conversion OPENMP_HAVE_WENUM_CONSTEXPR_CONVERSION_FLAG)
3736
check_cxx_compiler_flag(-Wextra OPENMP_HAVE_WEXTRA_FLAG)
3837
check_cxx_compiler_flag(-Wpedantic OPENMP_HAVE_WPEDANTIC_FLAG)
3938
check_cxx_compiler_flag(-Wmaybe-uninitialized OPENMP_HAVE_WMAYBE_UNINITIALIZED_FLAG)

0 commit comments

Comments
 (0)