Vulkan: Remove swiftshader workaround for missing gl_Position

The following line was added by the translator to work around a swiftshader bug where no gl_PerVertex declaration (and hence no gl_Position) would fail an assertion: gl_Position = gl_Position; The swiftshader bug is fixed, so this line can be removed. Bug: b/176161380 Change-Id: I68a8e0d9cd8b97e9d294a119c2daa4ab64ec863f Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/2633729Reviewed-by: Tim Van Patten <timvp@google.com> Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org>

Vulkan: Remove swiftshader workaround for missing gl_Position
203154b5 · Shahbaz Youssefi · Commit Bot · f28a06ba · 203154b5
Commit 203154b5 authored Jan 18, 2021 by Shahbaz Youssefi Committed by Commit Bot Jan 21, 2021
Hide whitespace changes
Inline Side-by-side

Showing with 0 additions and 11 deletions

TranslatorVulkan.cpp src/compiler/translator/TranslatorVulkan.cpp +0 -11

No files found.
--- a/src/compiler/translator/TranslatorVulkan.cpp
+++ b/src/compiler/translator/TranslatorVulkan.cpp
@@ -1148,17 +1148,6 @@ bool TranslatorVulkan::translateImpl(TIntermBlock *root,
                return false;
            }

-            // Work around a bug in SwiftShader where missing gl_Position causes an assertion error.
-            // This code appends gl_Position = gl_Position; to the end of the shader.
-            // b/176161380
-            TIntermSymbol *position = new TIntermSymbol(BuiltInVariable::gl_Position());
-            TIntermBinary *assignment =
-                new TIntermBinary(TOperator::EOpAssign, position, position->deepCopy());
-            if (!RunAtTheEndOfShader(this, root, assignment, &getSymbolTable()))
-            {
-                return false;
-            }
-
            break;
        }