<p dir="ltr"><br>
On Nov 25, 2015 1:27 PM, "Emil Velikov" <<a href="mailto:emil.l.velikov@gmail.com">emil.l.velikov@gmail.com</a>> wrote:<br>
><br>
> From: Emil Velikov <<a href="mailto:emil.velikov@collabora.com">emil.velikov@collabora.com</a>><br>
><br>
> ... alongside its C++ brethren. On the flip side the files are called<br>
> nir_types.{cpp,h} just because...</p>
<p dir="ltr">It does do one NIR-specific thing: if you call glsl_get_length on a matrix, it gives you the number of columns.  This probably isn't a huge deal though.</p>
<p dir="ltr">> At some point in the future we can fold the wrappers where they should<br>
> be and rename things appropriately. Yet current direction is to try<br>
> and split it out of nir.<br>
><br>
> Signed-off-by: Emil Velikov <<a href="mailto:emil.velikov@collabora.com">emil.velikov@collabora.com</a>><br>
> ---<br>
>  src/compiler/Makefile.sources            | 2 ++<br>
>  src/{glsl/nir => compiler}/nir_types.cpp | 2 +-<br>
>  src/{glsl/nir => compiler}/nir_types.h   | 3 ++-<br>
>  src/glsl/Makefile.sources                | 4 +---<br>
>  src/glsl/nir/nir.h                       | 2 +-<br>
>  src/glsl/nir/nir_lower_var_copies.c      | 2 +-<br>
>  6 files changed, 8 insertions(+), 7 deletions(-)<br>
>  rename src/{glsl/nir => compiler}/nir_types.cpp (99%)<br>
>  rename src/{glsl/nir => compiler}/nir_types.h (98%)<br>
><br>
> diff --git a/src/compiler/Makefile.sources b/src/compiler/Makefile.sources<br>
> index 38e75cf..e1228ca 100644<br>
> --- a/src/compiler/Makefile.sources<br>
> +++ b/src/compiler/Makefile.sources<br>
> @@ -2,5 +2,7 @@ LIBCOMPILER_FILES = \<br>
>         builtin_type_macros.h \<br>
>         glsl_types.cpp \<br>
>         glsl_types.h \<br>
> +       nir_types.cpp \<br>
> +       nir_types.h \<br>
>         shader_enums.c \<br>
>         shader_enums.h<br>
> diff --git a/src/glsl/nir/nir_types.cpp b/src/compiler/nir_types.cpp<br>
> similarity index 99%<br>
> rename from src/glsl/nir/nir_types.cpp<br>
> rename to src/compiler/nir_types.cpp<br>
> index 135591a..4f89d20 100644<br>
> --- a/src/glsl/nir/nir_types.cpp<br>
> +++ b/src/compiler/nir_types.cpp<br>
> @@ -25,8 +25,8 @@<br>
>   *<br>
>   */<br>
><br>
> +#include "glsl/ir.h"<br>
>  #include "nir_types.h"<br>
> -#include "ir.h"<br>
><br>
>  void<br>
>  glsl_print_type(const glsl_type *type, FILE *fp)<br>
> diff --git a/src/glsl/nir/nir_types.h b/src/compiler/nir_types.h<br>
> similarity index 98%<br>
> rename from src/glsl/nir/nir_types.h<br>
> rename to src/compiler/nir_types.h<br>
> index 572cff2..b2245c2 100644<br>
> --- a/src/glsl/nir/nir_types.h<br>
> +++ b/src/compiler/nir_types.h<br>
> @@ -28,10 +28,11 @@<br>
>  #pragma once<br>
><br>
>  #include <stdio.h><br>
> +#include <stdbool.h><br>
><br>
>  /* C wrapper around compiler/glsl_types.h */<br>
><br>
> -#include "compiler/glsl_types.h"<br>
> +#include "glsl_types.h"<br>
><br>
>  #ifdef __cplusplus<br>
>  extern "C" {<br>
> diff --git a/src/glsl/Makefile.sources b/src/glsl/Makefile.sources<br>
> index 80cbd76..5bf6486 100644<br>
> --- a/src/glsl/Makefile.sources<br>
> +++ b/src/glsl/Makefile.sources<br>
> @@ -76,12 +76,10 @@ NIR_FILES = \<br>
>         nir/nir_split_var_copies.c \<br>
>         nir/nir_sweep.c \<br>
>         nir/nir_to_ssa.c \<br>
> -       nir/nir_types.h \<br>
>         nir/nir_validate.c \<br>
>         nir/nir_vla.h \<br>
>         nir/nir_worklist.c \<br>
> -       nir/nir_worklist.h \<br>
> -       nir/nir_types.cpp<br>
> +       nir/nir_worklist.h<br>
><br>
>  # libglsl<br>
><br>
> diff --git a/src/glsl/nir/nir.h b/src/glsl/nir/nir.h<br>
> index e69b3c8..7e99a7b 100644<br>
> --- a/src/glsl/nir/nir.h<br>
> +++ b/src/glsl/nir/nir.h<br>
> @@ -34,7 +34,7 @@<br>
>  #include "util/ralloc.h"<br>
>  #include "util/set.h"<br>
>  #include "util/bitset.h"<br>
> -#include "nir_types.h"<br>
> +#include "compiler/nir_types.h"<br>
>  #include "compiler/shader_enums.h"<br>
>  #include <stdio.h><br>
><br>
> diff --git a/src/glsl/nir/nir_lower_var_copies.c b/src/glsl/nir/nir_lower_var_copies.c<br>
> index 98c107a..f5a5815 100644<br>
> --- a/src/glsl/nir/nir_lower_var_copies.c<br>
> +++ b/src/glsl/nir/nir_lower_var_copies.c<br>
> @@ -25,8 +25,8 @@<br>
>   *<br>
>   */<br>
><br>
> +#include "compiler/nir_types.h"<br>
>  #include "nir.h"<br>
> -#include "nir_types.h"<br>
><br>
>  /*<br>
>   * Lowers all copy intrinsics to sequences of load/store intrinsics.<br>
> --<br>
> 2.6.2<br>
><br>
> _______________________________________________<br>
> mesa-dev mailing list<br>
> <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</p>