On Sep 26, 2016 3:00 AM, "Eric Engestrom" <eric.engest...@imgtec.com> wrote:
>
> On Sun, Sep 25, 2016 at 10:49:29AM -0700, Jason Ekstrand wrote:
> > I hope you realize that this is the only truly useful change in the
series.
> > :-). Still, no reason why our silly little helpers shouldn't be correct.
>
> Yeah, I know :P
> I got the Coverity report like everyone and thought we might as well
> print real error messages, esp. since asserts are gone in release builds
> (but who would use spirv2nir in a release build? ^^)
>
> > Series is
> >
> > Reviewed-by: Jason Ekstrand <ja...@jlekstrand.net>
>
> Thanks! Can you push it for me?

Pushed. Thanks!

> Cheers,
>   Eric
>
> >
> > On Sep 25, 2016 6:50 PM, "Eric Engestrom" <e...@engestrom.ch> wrote:
> >
> > > Signed-off-by: Eric Engestrom <e...@engestrom.ch>
> > > ---
> > >  src/compiler/.gitignore | 1 +
> > >  1 file changed, 1 insertion(+)
> > >
> > > diff --git a/src/compiler/.gitignore b/src/compiler/.gitignore
> > > index c0e6299..c4f17be 100644
> > > --- a/src/compiler/.gitignore
> > > +++ b/src/compiler/.gitignore
> > > @@ -3,3 +3,4 @@ subtest-cr
> > >  subtest-cr-lf
> > >  subtest-lf
> > >  subtest-lf-cr
> > > +spirv2nir
> > > --
> > > Cheers,
> > >   Eric
> > >
> > >
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to