On 05/09/2016 03:45 PM, Jose Fonseca wrote:
This reverts commit 99474dc29b0f45413d0e538d7321da9ad4c9f6f5.

-Wpedantic is too verbose, even when applied to a few includes.

We'll just have to deal with the issues as they come.
---
  src/compiler/nir/glsl_to_nir.cpp | 23 +----------------------
  1 file changed, 1 insertion(+), 22 deletions(-)

diff --git a/src/compiler/nir/glsl_to_nir.cpp b/src/compiler/nir/glsl_to_nir.cpp
index c8954ce..e3fa623 100644
--- a/src/compiler/nir/glsl_to_nir.cpp
+++ b/src/compiler/nir/glsl_to_nir.cpp
@@ -25,30 +25,9 @@
   *
   */

-#include "nir.h"
-
-/*
- * Have GCC/Clang warn when NIR headers use C99 extensions that are not
- * supported in C++, as they are not portable.  In particular MSVC does not
- * support C99 structure field initializers.
- */
-#if defined(__GNUC__)
-#  pragma GCC diagnostic push
-#  if defined(__clang__)
-#    pragma GCC diagnostic warning "-Wc99-extensions"
-#  else
-#    pragma GCC diagnostic warning "-Wpedantic"
-#  endif
-#endif
-
+#include "glsl_to_nir.h"
  #include "nir_control_flow.h"
  #include "nir_builder.h"
-
-#if defined(__GNUC__)
-#  pragma GCC diagnostic pop
-#endif
-
-#include "glsl_to_nir.h"
  #include "compiler/glsl/ir_visitor.h"
  #include "compiler/glsl/ir_hierarchical_visitor.h"
  #include "compiler/glsl/ir.h"


Reviewed-by: Brian Paul <[email protected]>

_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to