<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body><table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>Bug ID</th>
<td><a class="bz_bug_link
bz_status_NEW "
title="NEW - [regression, bisected] arb_shader_image_load_store.compiler.declaration-format-qualifier-duplicate"
href="https://bugs.freedesktop.org/show_bug.cgi?id=93783">93783</a>
</td>
</tr>
<tr>
<th>Summary</th>
<td>[regression, bisected] arb_shader_image_load_store.compiler.declaration-format-qualifier-duplicate
</td>
</tr>
<tr>
<th>Product</th>
<td>Mesa
</td>
</tr>
<tr>
<th>Version</th>
<td>git
</td>
</tr>
<tr>
<th>Hardware</th>
<td>x86-64 (AMD64)
</td>
</tr>
<tr>
<th>OS</th>
<td>Linux (All)
</td>
</tr>
<tr>
<th>Status</th>
<td>NEW
</td>
</tr>
<tr>
<th>Severity</th>
<td>normal
</td>
</tr>
<tr>
<th>Priority</th>
<td>medium
</td>
</tr>
<tr>
<th>Component</th>
<td>Drivers/DRI/i965
</td>
</tr>
<tr>
<th>Assignee</th>
<td>idr@freedesktop.org
</td>
</tr>
<tr>
<th>Reporter</th>
<td>mark.a.janes@intel.com
</td>
</tr>
<tr>
<th>QA Contact</th>
<td>intel-3d-bugs@lists.freedesktop.org
</td>
</tr></table>
<p>
<div>
<pre>arb_shader_image_load_store.compiler.declaration-format-qualifier-duplicate.frag
and
arb_shader_image_load_store.compiler.declaration-format-qualifier-duplicate.vert
regressed on mesa 6a660a5f5dad02a6594ea905c511ba3cae6862a5
glsl: allow multiple layout qualifiers for a single declaration
From the ARB_shading_language_420pack spec:
"More than one layout qualifier may appear in a single
declaration. If the same layout-qualifier-name occurs in
multiple layout qualifiers for the same declaration, the
last one overrides the former ones."
The parser was already failing correctly when the extension is
not available but testing for duplicates within a single layout
qualifier was still causing this to fail when available as both
cases share the same function for merging.
Here we add a parameter to differentiate between the two uses
and apply it to the duplicate test.
However, the piglit test's output references a different part of the spec in
asserting that multiple layout qualifiers are not permitted:
/tmp/build_root/m64/lib/piglit/bin/glslparsertest
/tmp/build_root/m64/lib/piglit/generated_tests/spec/ARB_shader_image_load_store/compiler/declaration-format-qualifier-duplicate.frag
fail 1.50 GL_ARB_shader_image_load_store
piglit: debug: Requested an OpenGL 3.2 Core Context, and received a matching
3.3 context
Shader source:
/*
* [config]
* expect_result: fail
* glsl_version: 1.50
* require_extensions: GL_ARB_shader_image_load_store
* [end config]
*/
#version 150
#extension GL_ARB_shader_image_load_store: require
/*
* From the ARB_shader_image_load_store spec:
*
* "Only one format qualifier may be specified for any image variable
* declaration."
*/
layout(rgba32f) layout(rgba32f) uniform image2D img;
void main()
{
}
Standard Error
Successfully compiled fragment shader
/tmp/build_root/m64/lib/piglit/generated_tests/spec/ARB_shader_image_load_store/compiler/declaration-format-qualifier-duplicate.frag:</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the QA Contact for the bug.</li>
</ul>
</body>
</html>