Re: [Mesa-dev] [RFCv3 04/11] nir: allow pre-resolved sampler uniform locations

2016-03-22 Thread Rob Clark
So I've been looking a bit more at this, and skipping the shader_program->UniformStorage[location].opaque[stage].index offset is, I think, not the right thing to do. The original issue I was having with this seems to be a disagreement about location idx into UniformStorage. In parcel_out_uniform_

[Mesa-dev] [RFCv3 04/11] nir: allow pre-resolved sampler uniform locations

2016-01-31 Thread Rob Clark
From: Rob Clark With TGSI, the ir_variable::data.location gets fixed up to be a stage local location (rather than program global). In this case we need to skip the UniformStorage[location] lookup. Signed-off-by: Rob Clark --- src/compiler/nir/nir_lower_samplers.c | 23 ---