On 26.10.2016 02:59, Ian Romanick wrote:
From: Ian Romanick <ian.d.roman...@intel.com>
This is only in libstandalone currently because it will only be used in
the stand-alone compiler.
v2: Change the signature of the generated function. The ir_factory is
created in the generator, and an availability predicate is taken as a
parameter.
Signed-off-by: Ian Romanick <ian.d.roman...@intel.com>
---
src/compiler/Makefile.sources | 2 +
src/compiler/glsl/ir_builder_print_visitor.cpp | 756 +++++++++++++++++++++++++
src/compiler/glsl/ir_builder_print_visitor.h | 32 ++
3 files changed, 790 insertions(+)
create mode 100644 src/compiler/glsl/ir_builder_print_visitor.cpp
create mode 100644 src/compiler/glsl/ir_builder_print_visitor.h
diff --git a/src/compiler/Makefile.sources b/src/compiler/Makefile.sources
index a30443d..9f2d362 100644
--- a/src/compiler/Makefile.sources
+++ b/src/compiler/Makefile.sources
@@ -142,6 +142,8 @@ LIBGLSL_SHADER_CACHE_FILES = \
# glsl_compiler
GLSL_COMPILER_CXX_FILES = \
+ glsl/ir_builder_print_visitor.cpp \
+ glsl/ir_builder_print_visitor.h \
glsl/standalone_scaffolding.cpp \
glsl/standalone_scaffolding.h \
glsl/standalone.cpp \
diff --git a/src/compiler/glsl/ir_builder_print_visitor.cpp
b/src/compiler/glsl/ir_builder_print_visitor.cpp
new file mode 100644
index 0000000..d7bba9b
--- /dev/null
+++ b/src/compiler/glsl/ir_builder_print_visitor.cpp
[snip]
+ir_visitor_status
+ir_builder_print_visitor::visit(ir_variable *ir)
+{
+ const unsigned my_index = next_ir_index++;
+
+ _mesa_hash_table_insert(index_map, ir, (void *)(uintptr_t) my_index);
+
+ const char *mode_str;
+ switch (ir->data.mode) {
+ case ir_var_auto: mode_str = "ir_var_auto"; break;
+ case ir_var_uniform: mode_str = "ir_var_uniform"; break;
+ case ir_var_shader_storage: mode_str = "ir_var_shader_storage"; break;
+ case ir_var_shader_shared: mode_str = "ir_var_shader_shared"; break;
+ case ir_var_shader_in: mode_str = "ir_var_shader_in"; break;
+ case ir_var_shader_out: mode_str = "ir_var_shader_out"; break;
+ case ir_var_function_in: mode_str = "ir_var_function_in"; break;
+ case ir_var_function_out: mode_str = "ir_var_function_out"; break;
+ case ir_var_function_inout: mode_str = "ir_var_function_inout"; break;
+ case ir_var_const_in: mode_str = "ir_var_const_in"; break;
+ case ir_var_system_value: mode_str = "ir_var_system_value"; break;
+ case ir_var_temporary: mode_str = "ir_var_temporary"; break;
+ default:
+ unreachable("Invalid variable mode");
+ }
+
+ if (ir->data.mode == ir_var_temporary) {
+ print_with_indent("ir_variable *const r%04X = body.make_temp(glsl_type::%s_type,
\"%s\");\n",
+ my_index,
+ ir->type->name,
I take it this is not meant to be used for code with more complex types?
Apart from that, this pair of patches looks reasonable; 5 & 6:
Reviewed-by: Nicolai Hähnle <nicolai.haeh...@amd.com>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
