On 11/18/2011 11:27 AM, Marek Olšák wrote: > diff --git a/src/glsl/ir_clone.cpp b/src/glsl/ir_clone.cpp > index e8ac9fb..c63615c 100644 > --- a/src/glsl/ir_clone.cpp > +++ b/src/glsl/ir_clone.cpp > @@ -51,6 +51,7 @@ ir_variable::clone(void *mem_ctx, struct hash_table *ht) > const > var->pixel_center_integer = this->pixel_center_integer; > var->explicit_location = this->explicit_location; > var->has_initializer = this->has_initializer; > + var->depth_layout = this->depth_layout; > > var->num_state_slots = this->num_state_slots; > if (this->state_slots) {
This looks like a useful hunk that we must've missed. It's also fairly unrelated to the rest of your patch (splitting AMD/ARB enable bits). I don't think we need to split the AMD/ARB enable bits; it's the exact same extension with a name change and some rewording of the spec language. I'd be in favor of pushing this hunk as it's own patch and dropping the rest. You can have my R-b on such a patch. _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev