From 1ebf3c4b6741a3a3a9d46048abe3996cb9a86334 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Marek=20Ol=C5=A1=C3=A1k?= Date: Mon, 8 Aug 2016 22:05:29 +0200 Subject: [PATCH] Revert "glsl: don't try to lower non-gl builtins as if they were gl_FragData" This reverts commit a37e46323c7e18bec4160f2f66847c10b7041dc1. It broke the game Overlord such that it hung a GCN GNU. While I don't know how the hang happened because of its randomness and gfx corruption precedes it, many of the shaders contain this: out vec4 FragData[gl_MaxDrawBuffers]; --- src/compiler/glsl/opt_dead_builtin_varyings.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/compiler/glsl/opt_dead_builtin_varyings.cpp b/src/compiler/glsl/opt_dead_builtin_varyings.cpp index 900a096..33648d7 100644 --- a/src/compiler/glsl/opt_dead_builtin_varyings.cpp +++ b/src/compiler/glsl/opt_dead_builtin_varyings.cpp @@ -85,8 +85,7 @@ public: { ir_variable *var = ir->variable_referenced(); - if (!var || var->data.mode != this->mode || !var->type->is_array() || - !is_gl_identifier(var->name)) + if (!var || var->data.mode != this->mode || !var->type->is_array()) return visit_continue; /* Only match gl_FragData[], not gl_SecondaryFragDataEXT[] */ -- 2.7.4