On Sep 26, 2016 3:00 AM, "Eric Engestrom" <<a href="mailto:eric.engestrom@imgtec.com">eric.engestrom@imgtec.com</a>> 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 <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> > > Thanks! Can you push it for me? Pushed. Thanks! > Cheers, > Eric > > > > > On Sep 25, 2016 6:50 PM, "Eric Engestrom" <<a href="mailto:eric@engestrom.ch">eric@engestrom.ch</a>> wrote: > > > > > Signed-off-by: Eric Engestrom <<a href="mailto:eric@engestrom.ch">eric@engestrom.ch</a>> > > > --- > > > 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 > > > > > >