[Mesa-dev] [PATCH v2 02/82] glsl: Add ir_var_shader_storage
Jordan Justen
jordan.l.justen at intel.com
Fri Jun 5 00:37:11 PDT 2015
On 2015-06-05 00:11:11, Iago Toral wrote:
> On Thu, 2015-06-04 at 00:12 -0700, Jordan Justen wrote:
> > On 2015-06-03 00:00:52, Iago Toral Quiroga wrote:
> > > From: Kristian Høgsberg <krh at bitplanet.net>
> (...)
> > > diff --git a/src/glsl/ir.h b/src/glsl/ir.h
> > > index fab1cd2..dd10298 100644
> > > --- a/src/glsl/ir.h
> > > +++ b/src/glsl/ir.h
> > > @@ -323,6 +323,7 @@ protected:
> > > enum ir_variable_mode {
> > > ir_var_auto = 0, /**< Function local variables and globals. */
> > > ir_var_uniform, /**< Variable declared as a uniform. */
> > > + ir_var_shader_storage, /**< Variable declared as an ssbo. */
> > > ir_var_shader_in,
> > > ir_var_shader_out,
> > > ir_var_function_in,
> > > @@ -444,7 +445,9 @@ public:
> > > */
> > > inline bool is_in_uniform_block() const
> >
> > How about renaming this? is_in_buffer_block? (shader storage and
> > uniform buffer interface blocks...)
>
> Sure, do you think we should do the rename in the same commit or
> separately?
Good point. Probably add the rename separately just after this patch.
You can add my r-b on that one too.
> > With that, Reviewed-by: Jordan Justen <jordan.l.justen at intel.com>
More information about the mesa-dev
mailing list