[Mesa-dev] [PATCH v2 13/18] compiler/lower_64bit_packing: rename the pass to be more generic

Jason Ekstrand jason at jlekstrand.net
Mon Apr 30 21:50:45 UTC 2018


Drp... Ignore my comments on 12...

Reviewed-by: Jason Ekstrand <jason at jlekstrand.net>

On Mon, Apr 30, 2018 at 7:18 AM, Iago Toral Quiroga <itoral at igalia.com>
wrote:

> It can do 32-bit packing too now.
> ---
>  src/amd/vulkan/radv_shader.c                                        | 2 +-
>  src/compiler/Makefile.sources                                       | 2 +-
>  src/compiler/nir/meson.build                                        | 2 +-
>  src/compiler/nir/nir.h                                              | 2 +-
>  src/compiler/nir/{nir_lower_64bit_packing.c => nir_lower_packing.c} | 6
> +++---
>  src/intel/compiler/brw_nir.c                                        | 2 +-
>  src/mesa/state_tracker/st_glsl_to_nir.cpp                           | 2
> +-
>  7 files changed, 9 insertions(+), 9 deletions(-)
>  rename src/compiler/nir/{nir_lower_64bit_packing.c =>
> nir_lower_packing.c} (97%)
>
> diff --git a/src/amd/vulkan/radv_shader.c b/src/amd/vulkan/radv_shader.c
> index aaa6702975..67956000d1 100644
> --- a/src/amd/vulkan/radv_shader.c
> +++ b/src/amd/vulkan/radv_shader.c
> @@ -125,7 +125,7 @@ radv_optimize_nir(struct nir_shader *shader)
>                  progress = false;
>
>                  NIR_PASS_V(shader, nir_lower_vars_to_ssa);
> -               NIR_PASS_V(shader, nir_lower_64bit_pack);
> +               NIR_PASS_V(shader, nir_lower_pack);
>                  NIR_PASS_V(shader, nir_lower_alu_to_scalar);
>                  NIR_PASS_V(shader, nir_lower_phis_to_scalar);
>
> diff --git a/src/compiler/Makefile.sources b/src/compiler/Makefile.sources
> index b5321588be..14d024b0f9 100644
> --- a/src/compiler/Makefile.sources
> +++ b/src/compiler/Makefile.sources
> @@ -208,7 +208,6 @@ NIR_FILES = \
>         nir/nir_liveness.c \
>         nir/nir_loop_analyze.c \
>         nir/nir_loop_analyze.h \
> -       nir/nir_lower_64bit_packing.c \
>         nir/nir_lower_alpha_test.c \
>         nir/nir_lower_alu_to_scalar.c \
>         nir/nir_lower_atomics.c \
> @@ -232,6 +231,7 @@ NIR_FILES = \
>         nir/nir_lower_io_to_temporaries.c \
>         nir/nir_lower_io_to_scalar.c \
>         nir/nir_lower_io_types.c \
> +       nir/nir_lower_packing.c \
>         nir/nir_lower_passthrough_edgeflags.c \
>         nir/nir_lower_patch_vertices.c \
>         nir/nir_lower_phis_to_scalar.c \
> diff --git a/src/compiler/nir/meson.build b/src/compiler/nir/meson.build
> index 62fe5167d3..1307e52f92 100644
> --- a/src/compiler/nir/meson.build
> +++ b/src/compiler/nir/meson.build
> @@ -103,7 +103,6 @@ files_libnir = files(
>    'nir_liveness.c',
>    'nir_loop_analyze.c',
>    'nir_loop_analyze.h',
> -  'nir_lower_64bit_packing.c',
>    'nir_lower_alu_to_scalar.c',
>    'nir_lower_alpha_test.c',
>    'nir_lower_atomics.c',
> @@ -127,6 +126,7 @@ files_libnir = files(
>    'nir_lower_io_to_temporaries.c',
>    'nir_lower_io_to_scalar.c',
>    'nir_lower_io_types.c',
> +  'nir_lower_packing.c',
>    'nir_lower_passthrough_edgeflags.c',
>    'nir_lower_patch_vertices.c',
>    'nir_lower_phis_to_scalar.c',
> diff --git a/src/compiler/nir/nir.h b/src/compiler/nir/nir.h
> index e7f2b145b3..d328d90b84 100644
> --- a/src/compiler/nir/nir.h
> +++ b/src/compiler/nir/nir.h
> @@ -2791,7 +2791,7 @@ typedef enum {
>  } nir_lower_doubles_options;
>
>  bool nir_lower_doubles(nir_shader *shader, nir_lower_doubles_options
> options);
> -bool nir_lower_64bit_pack(nir_shader *shader);
> +bool nir_lower_pack(nir_shader *shader);
>
>  bool nir_normalize_cubemap_coords(nir_shader *shader);
>
> diff --git a/src/compiler/nir/nir_lower_64bit_packing.c
> b/src/compiler/nir/nir_lower_packing.c
> similarity index 97%
> rename from src/compiler/nir/nir_lower_64bit_packing.c
> rename to src/compiler/nir/nir_lower_packing.c
> index dd435490e3..ba9f4bc040 100644
> --- a/src/compiler/nir/nir_lower_64bit_packing.c
> +++ b/src/compiler/nir/nir_lower_packing.c
> @@ -87,7 +87,7 @@ lower_unpack_64_to_16(nir_builder *b, nir_ssa_def *src)
>  }
>
>  static bool
> -lower_64bit_pack_impl(nir_function_impl *impl)
> +lower_pack_impl(nir_function_impl *impl)
>  {
>     nir_builder b;
>     nir_builder_init(&b, impl);
> @@ -148,13 +148,13 @@ lower_64bit_pack_impl(nir_function_impl *impl)
>  }
>
>  bool
> -nir_lower_64bit_pack(nir_shader *shader)
> +nir_lower_pack(nir_shader *shader)
>  {
>     bool progress = false;
>
>     nir_foreach_function(function, shader) {
>        if (function->impl)
> -         progress |= lower_64bit_pack_impl(function->impl);
> +         progress |= lower_pack_impl(function->impl);
>     }
>
>     return false;
> diff --git a/src/intel/compiler/brw_nir.c b/src/intel/compiler/brw_nir.c
> index fb5e08fb33..22ef0486b7 100644
> --- a/src/intel/compiler/brw_nir.c
> +++ b/src/intel/compiler/brw_nir.c
> @@ -584,7 +584,7 @@ brw_nir_optimize(nir_shader *nir, const struct
> brw_compiler *compiler,
>                               nir_lower_dfract |
>                               nir_lower_dround_even |
>                               nir_lower_dmod);
> -      OPT(nir_lower_64bit_pack);
> +      OPT(nir_lower_pack);
>     } while (progress);
>
>     return nir;
> diff --git a/src/mesa/state_tracker/st_glsl_to_nir.cpp
> b/src/mesa/state_tracker/st_glsl_to_nir.cpp
> index bcf6a7ceb6..dbf506e77b 100644
> --- a/src/mesa/state_tracker/st_glsl_to_nir.cpp
> +++ b/src/mesa/state_tracker/st_glsl_to_nir.cpp
> @@ -324,7 +324,7 @@ st_nir_opts(nir_shader *nir)
>        NIR_PASS_V(nir, nir_lower_alu_to_scalar);
>        NIR_PASS_V(nir, nir_lower_phis_to_scalar);
>
> -      NIR_PASS_V(nir, nir_lower_64bit_pack);
> +      NIR_PASS_V(nir, nir_lower_pack);
>        NIR_PASS(progress, nir, nir_copy_prop);
>        NIR_PASS(progress, nir, nir_opt_remove_phis);
>        NIR_PASS(progress, nir, nir_opt_dce);
> --
> 2.14.1
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20180430/2bf186cd/attachment-0001.html>


More information about the mesa-dev mailing list