On Wed, Sep 30, 2015 at 10:08 AM, Matt Turner <matts...@gmail.com> wrote: > On Wed, Sep 30, 2015 at 9:59 AM, Kenneth Graunke <kenn...@whitecape.org> > wrote: >> Jason open coded this in 60befc63 when cleaning up some ugly code; >> using our existing helper tidies it up a bit more.
Sorry about that. Reviewed-by: Jason Ekstrand <jason.ekstr...@intel.com> >> Signed-off-by: Kenneth Graunke <kenn...@whitecape.org> >> --- >> 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 <matts...@gmail.com> > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
