Hash :
474a08c0
Author :
Date :
2016-06-28T10:49:46
Remove qualifiers from empty struct declarations in output
Empty struct declarations with qualifiers are rejected by NVIDIA GL
driver version 367.27. For example this kind of construct that is
expected to be accepted by the WebGL conformance tests is rejected:
const struct a {
int i;
};
Since qualifiers do not carry meaning unless a struct declaration
has declarators, they can be simply omitted from the translator
output in this kind of cases to work around this driver issue.
New unit test is added to check that pruning empty declarations works
correctly.
BUG=angleproject:1430
BUG=622492
TEST=WebGL conformance tests, angle_unittests
Change-Id: Id83f83124ae597fcdfa15100d336c2c207d9449c
Reviewed-on: https://chromium-review.googlesource.com/356362
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Commit-Queue: Olli Etuaho <oetuaho@nvidia.com>
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58
//
// Copyright (c) 2016 The ANGLE Project Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
//
// PruneEmptyDeclarations_test.cpp:
// Tests for pruning empty declarations.
//
#include "angle_gl.h"
#include "gtest/gtest.h"
#include "GLSLANG/ShaderLang.h"
#include "tests/test_utils/compiler_test.h"
namespace
{
class PruneEmptyDeclarationsTest : public MatchOutputCodeTest
{
public:
PruneEmptyDeclarationsTest()
: MatchOutputCodeTest(GL_VERTEX_SHADER, 0, SH_GLSL_COMPATIBILITY_OUTPUT)
{
}
};
TEST_F(PruneEmptyDeclarationsTest, EmptyDeclarationStartsDeclaratorList)
{
const std::string shaderString =
"precision mediump float;\n"
"uniform float u;\n"
"void main()\n"
"{\n"
" float, f;\n"
" gl_Position = vec4(u * f);\n"
"}\n";
compile(shaderString);
ASSERT_TRUE(foundInCode("float f"));
ASSERT_TRUE(notFoundInCode("float, f"));
}
TEST_F(PruneEmptyDeclarationsTest, EmptyStructDeclarationWithQualifiers)
{
const std::string shaderString =
"precision mediump float;\n"
"const struct S { float f; };\n"
"uniform S s;"
"void main()\n"
"{\n"
" gl_Position = vec4(s.f);\n"
"}\n";
compile(shaderString);
ASSERT_TRUE(foundInCode("struct S"));
ASSERT_TRUE(foundInCode("uniform S"));
ASSERT_TRUE(notFoundInCode("const struct S"));
}
} // namespace