On Sun, 2015-09-13 at 11:51 -0400, Rob Clark wrote: > From: Rob Clark <robcl...@freedesktop.org> > > Signed-off-by: Rob Clark <robcl...@freedesktop.org> > --- > src/Makefile.am | 1 + > src/glsl/shader_enums.c | 202 > ++++++++++++++++++++++++++++++++++++++++++++++ > src/glsl/shader_enums.h | 53 ++++++++++++ > src/mesa/Makefile.sources | 4 +- > 4 files changed, 259 insertions(+), 1 deletion(-) > create mode 100644 src/glsl/shader_enums.c > > diff --git a/src/Makefile.am b/src/Makefile.am > index 0d49bcd..9e15cca 100644 > --- a/src/Makefile.am > +++ b/src/Makefile.am > @@ -62,6 +62,7 @@ AM_CPPFLAGS = \ > noinst_LTLIBRARIES = libglsl_util.la > > libglsl_util_la_SOURCES = \ > + glsl/shader_enums.c \ > mesa/main/imports.c \ > mesa/program/prog_hash_table.c \ > mesa/program/symbol_table.c \ > diff --git a/src/glsl/shader_enums.c b/src/glsl/shader_enums.c > new file mode 100644 > index 0000000..1c45a40 > --- /dev/null > +++ b/src/glsl/shader_enums.c > @@ -0,0 +1,202 @@ > +/* > + * Mesa 3-D graphics library > + * > + * Copyright © 2015 Red Hat > + * > + * Permission is hereby granted, free of charge, to any person obtaining a > + * copy of this software and associated documentation files (the "Software"), > + * to deal in the Software without restriction, including without limitation > + * the rights to use, copy, modify, merge, publish, distribute, sublicense, > + * and/or sell copies of the Software, and to permit persons to whom the > + * Software is furnished to do so, subject to the following conditions: > + * > + * The above copyright notice and this permission notice (including the next > + * paragraph) shall be included in all copies or substantial portions of the > + * Software. > + * > + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR > + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, > + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL > + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER > + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING > + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER > DEALINGS > + * IN THE SOFTWARE. > + * > + * Authors: > + * Rob Clark <robcl...@freedesktop.org> > + */ > + > +#include "glsl/shader_enums.h" > +#include "util/macros.h" > + > +#define ENUM(x) [x] = #x > +#define NAME(val) ((((val) < ARRAY_SIZE(names)) && names[(val)]) ? > names[(val)] : "UNKNOWN") > + > +const char * gl_shader_stage_name(gl_shader_stage stage) > +{ > + static const char *names[] = { > + ENUM(MESA_SHADER_VERTEX), > + ENUM(MESA_SHADER_TESS_CTRL), > + ENUM(MESA_SHADER_TESS_EVAL), > + ENUM(MESA_SHADER_GEOMETRY), > + ENUM(MESA_SHADER_FRAGMENT), > + ENUM(MESA_SHADER_COMPUTE), > + }; > + return NAME(stage); > +} > + > +const char * gl_vert_attrib_name(gl_vert_attrib attrib) > +{ > + static const char *names[] = { > + ENUM(VERT_ATTRIB_POS), > + ENUM(VERT_ATTRIB_WEIGHT), > + ENUM(VERT_ATTRIB_NORMAL), > + ENUM(VERT_ATTRIB_COLOR0), > + ENUM(VERT_ATTRIB_COLOR1), > + ENUM(VERT_ATTRIB_FOG), > + ENUM(VERT_ATTRIB_COLOR_INDEX), > + ENUM(VERT_ATTRIB_EDGEFLAG), > + ENUM(VERT_ATTRIB_TEX0), > + ENUM(VERT_ATTRIB_TEX1), > + ENUM(VERT_ATTRIB_TEX2), > + ENUM(VERT_ATTRIB_TEX3), > + ENUM(VERT_ATTRIB_TEX4), > + ENUM(VERT_ATTRIB_TEX5), > + ENUM(VERT_ATTRIB_TEX6), > + ENUM(VERT_ATTRIB_TEX7), > + ENUM(VERT_ATTRIB_POINT_SIZE), > + ENUM(VERT_ATTRIB_GENERIC0), > + ENUM(VERT_ATTRIB_GENERIC1), > + ENUM(VERT_ATTRIB_GENERIC2), > + ENUM(VERT_ATTRIB_GENERIC3), > + ENUM(VERT_ATTRIB_GENERIC4), > + ENUM(VERT_ATTRIB_GENERIC5), > + ENUM(VERT_ATTRIB_GENERIC6), > + ENUM(VERT_ATTRIB_GENERIC7), > + ENUM(VERT_ATTRIB_GENERIC8), > + ENUM(VERT_ATTRIB_GENERIC9), > + ENUM(VERT_ATTRIB_GENERIC10), > + ENUM(VERT_ATTRIB_GENERIC11), > + ENUM(VERT_ATTRIB_GENERIC12), > + ENUM(VERT_ATTRIB_GENERIC13), > + ENUM(VERT_ATTRIB_GENERIC14), > + ENUM(VERT_ATTRIB_GENERIC15), > + }; > + return NAME(attrib); > +} > + > +const char * gl_varying_slot_name(gl_varying_slot slot) > +{ > + static const char *names[] = { > + ENUM(VARYING_SLOT_POS), > + ENUM(VARYING_SLOT_COL0), > + ENUM(VARYING_SLOT_COL1), > + ENUM(VARYING_SLOT_FOGC), > + ENUM(VARYING_SLOT_TEX0), > + ENUM(VARYING_SLOT_TEX1), > + ENUM(VARYING_SLOT_TEX2), > + ENUM(VARYING_SLOT_TEX3), > + ENUM(VARYING_SLOT_TEX4), > + ENUM(VARYING_SLOT_TEX5), > + ENUM(VARYING_SLOT_TEX6), > + ENUM(VARYING_SLOT_TEX7), > + ENUM(VARYING_SLOT_PSIZ), > + ENUM(VARYING_SLOT_BFC0), > + ENUM(VARYING_SLOT_BFC1), > + ENUM(VARYING_SLOT_EDGE), > + ENUM(VARYING_SLOT_CLIP_VERTEX), > + ENUM(VARYING_SLOT_CLIP_DIST0), > + ENUM(VARYING_SLOT_CLIP_DIST1), > + ENUM(VARYING_SLOT_PRIMITIVE_ID), > + ENUM(VARYING_SLOT_LAYER), > + ENUM(VARYING_SLOT_VIEWPORT), > + ENUM(VARYING_SLOT_FACE), > + ENUM(VARYING_SLOT_PNTC), > + ENUM(VARYING_SLOT_TESS_LEVEL_OUTER), > + ENUM(VARYING_SLOT_TESS_LEVEL_INNER), > + ENUM(VARYING_SLOT_VAR0), > + ENUM(VARYING_SLOT_VAR1), > + ENUM(VARYING_SLOT_VAR2), > + ENUM(VARYING_SLOT_VAR3), > + ENUM(VARYING_SLOT_VAR4), > + ENUM(VARYING_SLOT_VAR5), > + ENUM(VARYING_SLOT_VAR6), > + ENUM(VARYING_SLOT_VAR7), > + ENUM(VARYING_SLOT_VAR8), > + ENUM(VARYING_SLOT_VAR9), > + ENUM(VARYING_SLOT_VAR10), > + ENUM(VARYING_SLOT_VAR11), > + ENUM(VARYING_SLOT_VAR12), > + ENUM(VARYING_SLOT_VAR13), > + ENUM(VARYING_SLOT_VAR14), > + ENUM(VARYING_SLOT_VAR15), > + ENUM(VARYING_SLOT_VAR16), > + ENUM(VARYING_SLOT_VAR17), > + ENUM(VARYING_SLOT_VAR18), > + ENUM(VARYING_SLOT_VAR19), > + ENUM(VARYING_SLOT_VAR20), > + ENUM(VARYING_SLOT_VAR21), > + ENUM(VARYING_SLOT_VAR22), > + ENUM(VARYING_SLOT_VAR23), > + ENUM(VARYING_SLOT_VAR24), > + ENUM(VARYING_SLOT_VAR25), > + ENUM(VARYING_SLOT_VAR26), > + ENUM(VARYING_SLOT_VAR27), > + ENUM(VARYING_SLOT_VAR28), > + ENUM(VARYING_SLOT_VAR29), > + ENUM(VARYING_SLOT_VAR30), > + ENUM(VARYING_SLOT_VAR31), > + }; > + return NAME(slot); > +} > + > +const char * gl_system_value_name(gl_system_value sysval) > +{ > + static const char *names[] = { > + ENUM(SYSTEM_VALUE_VERTEX_ID), > + ENUM(SYSTEM_VALUE_INSTANCE_ID), > + ENUM(SYSTEM_VALUE_VERTEX_ID_ZERO_BASE), > + ENUM(SYSTEM_VALUE_BASE_VERTEX), > + ENUM(SYSTEM_VALUE_INVOCATION_ID), > + ENUM(SYSTEM_VALUE_FRONT_FACE), > + ENUM(SYSTEM_VALUE_SAMPLE_ID), > + ENUM(SYSTEM_VALUE_SAMPLE_POS), > + ENUM(SYSTEM_VALUE_SAMPLE_MASK_IN), > + ENUM(SYSTEM_VALUE_TESS_COORD), > + ENUM(SYSTEM_VALUE_VERTICES_IN), > + ENUM(SYSTEM_VALUE_PRIMITIVE_ID), > + ENUM(SYSTEM_VALUE_TESS_LEVEL_OUTER), > + ENUM(SYSTEM_VALUE_TESS_LEVEL_INNER), > + }; > + return NAME(sysval); > +} > + > +const char * glsl_interp_qualifier_name(enum glsl_interp_qualifier qual) > +{ > + static const char *names[] = { > + ENUM(INTERP_QUALIFIER_NONE), > + ENUM(INTERP_QUALIFIER_SMOOTH), > + ENUM(INTERP_QUALIFIER_FLAT), > + ENUM(INTERP_QUALIFIER_NOPERSPECTIVE), > + }; > + return NAME(qual); > +} > + > +const char * gl_frag_result_name(gl_frag_result result) > +{ > + static const char *names[] = { > + ENUM(FRAG_RESULT_DEPTH), > + ENUM(FRAG_RESULT_STENCIL), > + ENUM(FRAG_RESULT_COLOR), > + ENUM(FRAG_RESULT_SAMPLE_MASK), > + ENUM(FRAG_RESULT_DATA0), > + ENUM(FRAG_RESULT_DATA1), > + ENUM(FRAG_RESULT_DATA2), > + ENUM(FRAG_RESULT_DATA3), > + ENUM(FRAG_RESULT_DATA4), > + ENUM(FRAG_RESULT_DATA5), > + ENUM(FRAG_RESULT_DATA6), > + ENUM(FRAG_RESULT_DATA7), > + }; > + return NAME(result); > +} > diff --git a/src/glsl/shader_enums.h b/src/glsl/shader_enums.h > index 9bb163f..d054b87 100644 > --- a/src/glsl/shader_enums.h > +++ b/src/glsl/shader_enums.h > @@ -43,6 +43,8 @@ typedef enum > MESA_SHADER_COMPUTE = 5, > } gl_shader_stage; > > +const char * gl_shader_stage_name(gl_shader_stage stage); > + > #define MESA_SHADER_STAGES (MESA_SHADER_COMPUTE + 1) > > > @@ -91,6 +93,8 @@ typedef enum > VERT_ATTRIB_MAX = 33 > } gl_vert_attrib; > > +const char * gl_vert_attrib_name(gl_vert_attrib attrib); > + > /** > * Symbolic constats to help iterating over > * specific blocks of vertex attributes. > @@ -193,8 +197,43 @@ typedef enum > VARYING_SLOT_TESS_LEVEL_OUTER, /* Only appears as TCS output. */ > VARYING_SLOT_TESS_LEVEL_INNER, /* Only appears as TCS output. */ > VARYING_SLOT_VAR0, /* First generic varying slot */ > + /* the remaining are simply for the benefit of gl_varying_slot_name() > + * and not to be construed as an upper bound: > + */ > + VARYING_SLOT_VAR1, > + VARYING_SLOT_VAR2, > + VARYING_SLOT_VAR3, > + VARYING_SLOT_VAR4, > + VARYING_SLOT_VAR5, > + VARYING_SLOT_VAR6, > + VARYING_SLOT_VAR7, > + VARYING_SLOT_VAR8, > + VARYING_SLOT_VAR9, > + VARYING_SLOT_VAR10, > + VARYING_SLOT_VAR11, > + VARYING_SLOT_VAR12, > + VARYING_SLOT_VAR13, > + VARYING_SLOT_VAR14, > + VARYING_SLOT_VAR15, > + VARYING_SLOT_VAR16, > + VARYING_SLOT_VAR17, > + VARYING_SLOT_VAR18, > + VARYING_SLOT_VAR19, > + VARYING_SLOT_VAR20, > + VARYING_SLOT_VAR21, > + VARYING_SLOT_VAR22, > + VARYING_SLOT_VAR23, > + VARYING_SLOT_VAR24, > + VARYING_SLOT_VAR25, > + VARYING_SLOT_VAR26, > + VARYING_SLOT_VAR27, > + VARYING_SLOT_VAR28, > + VARYING_SLOT_VAR29, > + VARYING_SLOT_VAR30, > + VARYING_SLOT_VAR31, > } gl_varying_slot; > > +const char * gl_varying_slot_name(gl_varying_slot slot); > > /** > * Bitflags for varying slots. > @@ -366,6 +405,7 @@ typedef enum > SYSTEM_VALUE_MAX /**< Number of values */ > } gl_system_value; > > +const char * gl_system_value_name(gl_system_value sysval); > > /** > * The possible interpolation qualifiers that can be applied to a fragment > @@ -383,6 +423,8 @@ enum glsl_interp_qualifier > INTERP_QUALIFIER_COUNT /**< Number of interpolation qualifiers */ > }; > > +const char * glsl_interp_qualifier_name(enum glsl_interp_qualifier qual); > + > /** > * Fragment program results > */ > @@ -399,8 +441,19 @@ typedef enum > /* FRAG_RESULT_DATAn are the per-render-target (GLSL gl_FragData[n] > * or ARB_fragment_program fragment.color[n]) color results. If > * any are written, FRAG_RESULT_COLOR will not be written. > + * FRAG_RESULT_DATA1 and up are simply for the benefit of > + * gl_frag_result_name() and not to be construed as an upper bound > */ > FRAG_RESULT_DATA0 = 4, > + FRAG_RESULT_DATA1, > + FRAG_RESULT_DATA2, > + FRAG_RESULT_DATA3, > + FRAG_RESULT_DATA4, > + FRAG_RESULT_DATA5, > + FRAG_RESULT_DATA6, > + FRAG_RESULT_DATA7, > } gl_frag_result; > > +const char * gl_frag_result_name(gl_frag_result result); > + > #endif /* SHADER_ENUMS_H */ > diff --git a/src/mesa/Makefile.sources b/src/mesa/Makefile.sources > index ed9848c..2a719a0 100644 > --- a/src/mesa/Makefile.sources > +++ b/src/mesa/Makefile.sources > @@ -523,7 +523,9 @@ PROGRAM_FILES = \ > program/sampler.h \ > program/string_to_uint_map.cpp \ > program/symbol_table.c \ > - program/symbol_table.h > + program/symbol_table.h \ > + ../glsl/shader_enums.c \ > + ../glsl/shader_enums.h
Why not put these under LIBGLSL_FILES in src/glsl/Makefile.sources? That would seem like a more natural choice. Iago > PROGRAM_NIR_FILES = \ > program/prog_to_nir.c \ _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev