[Mesa-dev] [PATCH] nir/lower_int64: mark all metadata as dirty

Jason Ekstrand jason at jlekstrand.net
Mon Jul 16 15:20:19 UTC 2018


On Mon, Jul 16, 2018 at 7:30 AM Karol Herbst <kherbst at redhat.com> wrote:

> without this we might end up looping inside the dominator analysis
> infinitly. Hit by some 64 bit int div OpenCL CTS test.
>
> Signed-off-by: Karol Herbst <kherbst at redhat.com>
> ---
>  src/compiler/nir/nir_lower_int64.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/src/compiler/nir/nir_lower_int64.c
> b/src/compiler/nir/nir_lower_int64.c
> index 22f69ef4a60..09eac8cfc05 100644
> --- a/src/compiler/nir/nir_lower_int64.c
> +++ b/src/compiler/nir/nir_lower_int64.c
> @@ -279,6 +279,8 @@ lower_int64_impl(nir_function_impl *impl,
> nir_lower_int64_options options)
>        }
>     }
>
> +   impl->valid_metadata = nir_metadata_none;
>

Please use nir_metadata_preserve(impl, nir_metadata_none) instead.


> +
>     return progress;
>  }
>
> --
> 2.17.1
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20180716/816e335f/attachment.html>


More information about the mesa-dev mailing list