On Tuesday, May 27, 2014 07:49:04 PM Ian Romanick wrote: > From: Ian Romanick <ian.d.roman...@intel.com> > > This has the added perk that if you forget to set ir_type in the > constructor of a new subclass (or a new constructor of an existing > subclass) the compiler will tell you... instead of relying on > ir_validate or similar run-time detection. > > Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> > --- > src/glsl/ir.cpp | 65 > ++++++++++++++++++++++++++------------------------------- src/glsl/ir.h | > 46 +++++++++++++++++++++++----------------- > 2 files changed, 57 insertions(+), 54 deletions(-)
This seems reasonable to me. Patch 9 is: Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
