On Wed, Oct 7, 2015 at 3:11 PM, Matt Turner <matts...@gmail.com> wrote: > On Wed, Oct 7, 2015 at 7:11 AM, Kristian Høgsberg Kristensen > <k...@bitplanet.net> wrote: >> Signed-off-by: Kristian Høgsberg Kristensen <k...@bitplanet.net> >> --- >> src/mesa/drivers/dri/i965/brw_fs.cpp | 5 +++-- >> 1 file changed, 3 insertions(+), 2 deletions(-) >> >> diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp >> b/src/mesa/drivers/dri/i965/brw_fs.cpp >> index 65c3628..b79b4a4 100644 >> --- a/src/mesa/drivers/dri/i965/brw_fs.cpp >> +++ b/src/mesa/drivers/dri/i965/brw_fs.cpp >> @@ -5276,6 +5276,7 @@ brw_cs_emit(struct brw_context *brw, >> prog_data->local_size[2] = cp->LocalSize[2]; >> unsigned local_workgroup_size = >> cp->LocalSize[0] * cp->LocalSize[1] * cp->LocalSize[2]; >> + unsigned max_cs_threads = >> brw->intelScreen->compiler->devinfo->max_cs_threads; > > I'm not following how this helps? Seems like you want to just pass in > devinfo or something?
We want to pass in just brw_compiler eventually. At that point brw_context won't be available. Jason will do that next, and I guess you could argue that this patch belongs in that series. However, this one occurrence is the only place the compiler accesses devinfo values through brw_context and getting rid of it means callers without a real brw_context have to fake a little less. Kristian _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev