]> https://gitweb.dealii.org/ - dealii.git/commitdiff
unify check 10307/head
authorMatthias Maier <tamiko@43-1.org>
Fri, 22 May 2020 21:33:31 +0000 (16:33 -0500)
committerMatthias Maier <tamiko@43-1.org>
Sat, 23 May 2020 14:59:23 +0000 (09:59 -0500)
We had another DEAL_II_CONSTEXPR_BUG check in our CMake configuration.
Let's just use DEAL_II_CXX14_CONSTEXPR_BUG instead.

cmake/checks/check_03_compiler_bugs.cmake
include/deal.II/base/geometry_info.h
source/base/geometry_info.cc

index 5eb17500e81b3d71b81dcf8a654aa0393f55dac0..d090205882318378ce5c62cc7c32e46c5059e7d4 100644 (file)
@@ -33,22 +33,6 @@ IF(CMAKE_CXX_COMPILER_ID MATCHES "Intel" AND CMAKE_CXX_COMPILER_VERSION VERSION_
 ENDIF()
 
 
-#
-# gcc-4.8.1 has some problems with the constexpr "vertices_per_cell" in the
-# definition of alternating_form_at_vertices.
-#
-# TODO: Write a unit test.
-#
-# For now, just enable the workaround for Windows targets
-#
-# - Matthias Maier, 2013
-#
-IF( CMAKE_SYSTEM_NAME MATCHES "CYGWIN"
-    OR CMAKE_SYSTEM_NAME MATCHES "Windows" )
-  SET(DEAL_II_CONSTEXPR_BUG TRUE)
-ENDIF()
-
-
 #
 # Intel 16.0.1 produces wrong code that creates a race condition in
 # tests/fe/curl_curl_01.debug but 16.0.2 is known to work. Blacklist this
index 5fab2b0419f8a672cfa7e380e65cc66b21ac658b..00420b133e1bf79743a0f8cf58d976c52f427586 100644 (file)
@@ -2475,7 +2475,7 @@ struct GeometryInfo
   template <int spacedim>
   static void
   alternating_form_at_vertices
-#ifndef DEAL_II_CONSTEXPR_BUG
+#ifndef DEAL_II_CXX14_CONSTEXPR_BUG
     (const Point<spacedim> (&vertices)[vertices_per_cell],
      Tensor<spacedim - dim, spacedim> (&forms)[vertices_per_cell]);
 #else
@@ -4744,7 +4744,7 @@ template <int dim>
 template <int spacedim>
 inline void
 GeometryInfo<dim>::alternating_form_at_vertices
-#  ifndef DEAL_II_CONSTEXPR_BUG
+#  ifndef DEAL_II_CXX14_CONSTEXPR_BUG
   (const Point<spacedim> (&vertices)[vertices_per_cell],
    Tensor<spacedim - dim, spacedim> (&forms)[vertices_per_cell])
 #  else
index 6b70f9da80dfda01ae4b4b1541c48295eed33cdc..435cbd66f400248f1d6714c29a76a3c295f12cdb 100644 (file)
@@ -87,7 +87,7 @@ template struct GeometryInfo<4>;
 
 template void
 GeometryInfo<1>::alternating_form_at_vertices
-#ifndef DEAL_II_CONSTEXPR_BUG
+#ifndef DEAL_II_CXX14_CONSTEXPR_BUG
   (const Point<1> (&)[vertices_per_cell],
    Tensor<1 - 1, 1> (&)[vertices_per_cell])
 #else
@@ -97,7 +97,7 @@ GeometryInfo<1>::alternating_form_at_vertices
 
 template void
 GeometryInfo<1>::alternating_form_at_vertices
-#ifndef DEAL_II_CONSTEXPR_BUG
+#ifndef DEAL_II_CXX14_CONSTEXPR_BUG
   (const Point<2> (&)[vertices_per_cell],
    Tensor<2 - 1, 2> (&)[vertices_per_cell])
 #else
@@ -107,7 +107,7 @@ GeometryInfo<1>::alternating_form_at_vertices
 
 template void
 GeometryInfo<2>::alternating_form_at_vertices
-#ifndef DEAL_II_CONSTEXPR_BUG
+#ifndef DEAL_II_CXX14_CONSTEXPR_BUG
   (const Point<2> (&vertices)[vertices_per_cell],
    Tensor<2 - 2, 2> (&forms)[vertices_per_cell])
 #else
@@ -117,7 +117,7 @@ GeometryInfo<2>::alternating_form_at_vertices
 
 template void
 GeometryInfo<2>::alternating_form_at_vertices
-#ifndef DEAL_II_CONSTEXPR_BUG
+#ifndef DEAL_II_CXX14_CONSTEXPR_BUG
   (const Point<3> (&vertices)[vertices_per_cell],
    Tensor<3 - 2, 3> (&forms)[vertices_per_cell])
 #else
@@ -128,7 +128,7 @@ GeometryInfo<2>::alternating_form_at_vertices
 
 template void
 GeometryInfo<3>::alternating_form_at_vertices
-#ifndef DEAL_II_CONSTEXPR_BUG
+#ifndef DEAL_II_CXX14_CONSTEXPR_BUG
   (const Point<3> (&vertices)[vertices_per_cell],
    Tensor<3 - 3, 3> (&forms)[vertices_per_cell])
 #else

In the beginning the Universe was created. This has made a lot of people very angry and has been widely regarded as a bad move.

Douglas Adams


Typeset in Trocchi and Trocchi Bold Sans Serif.