On Wed, Sep 30, 2015 at 9:59 AM, Kenneth Graunke <[email protected]> wrote: > Jason open coded this in 60befc63 when cleaning up some ugly code; > using our existing helper tidies it up a bit more. > > Signed-off-by: Kenneth Graunke <[email protected]> > --- > src/mesa/drivers/dri/i965/brw_shader.cpp | 51 > +++++++++++++------------------- > 1 file changed, 20 insertions(+), 31 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_shader.cpp > b/src/mesa/drivers/dri/i965/brw_shader.cpp > index 785cb27..cdc7819 100644 > --- a/src/mesa/drivers/dri/i965/brw_shader.cpp > +++ b/src/mesa/drivers/dri/i965/brw_shader.cpp > @@ -72,6 +72,20 @@ shader_perf_log_mesa(void *data, const char *fmt, ...) > va_end(args); > } > > +static inline bool
I don't think you want inline here (it's static and only used in this file, so the compiler can decide what to do), but otherwise Reviewed-by: Matt Turner <[email protected]> _______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
