[Mesa-dev] undefined behaviour in spirv_to_nir.c

Karol Herbst kherbst at redhat.com
Fri May 17 06:24:36 UTC 2019


ohhh, yeah.. I think we can actually just remove that code, as it
shouldn't have any affect on the constants value.

On Fri, May 17, 2019 at 4:07 AM Jason Ekstrand <jason at jlekstrand.net> wrote:
>
> I think it's fine but I'm not at my computer right now.
>
> --Jason
>
> On May 16, 2019 20:58:03 Dave Airlie <airlied at gmail.com> wrote:
>
> > Coverity gave me this:
> >
> > mesa-19.1.0-rc2/src/compiler/spirv/spirv_to_nir.c:1987:
> > overlapping_assignment: Assigning "src[1][i].u8" to "src[1][i].u32",
> > which have overlapping memory locations and different types.
> >
> > and the following lines, I think it's actually undefined behaviour wrt
> > the C spec.
> >
> > Dave.
>
>
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list