URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=6f3caafe0246e8e4f822cbfdf1cb1e7efe960f57
Author: Ian Romanick <[email protected]>
Date: Tue Jan 22 01:14:36 2013 -0500
glsl: Remove ir_variable::uniform_block
v2: A previous patch contained a spurious hunk that removed an
assignment to ir_variable::uniform_block. That hunk was moved to this
patch. Suggested by Carl Worth.
Signed-off-by: Ian Romanick <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=47c92a6d0d94c6d4ac7102de4b66d5b3a898fb08
Author: Ian Romanick <[email protected]>
Date: Mon Jan 21 23:19:25 2013 -0500
glsl: Allow dereferencing fields of an interface instance
Signed-off-by: Ian Romanick <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=81299e709398294a4da8d682e433662741b14ac6
Author: Ian Romanick <[email protected]>
Date: Sat Dec 15 14:49:53 2012 -0800
glsl: Allow elimination of uniform block members
glGetActiveUniform is not supposed to report block members that are not
active even if they are included in the layout of the block. The block
layout is determined from the GLSL_TYPE_INTERFACE that defines the
block, so eliminating the ir_variables that correspond to the individual
fields is safe.
Fixes gles3conform test
uniform_buffer_object_getuniformindices_for_for_nonexistent_or_not_active_uniform_names.
This also fixes the assertion failures (added in the previous commit) in
gles3conform uniform_buffer_object_index_of_not_active_block,
uniform_buffer_object_inherit_and_override_layouts, and
uniform_buffer_object_repeat_global_scope_layouts.
Signed-off-by: Ian Romanick <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=3de6e4436140611fa69c102191e3f786301cc869
Author: Ian Romanick <[email protected]>
Date: Tue Jan 22 01:09:16 2013 -0500
glsl: Calculate UBO data at link-time
Use the function added in the previous commit.
This temporarily causes gles3conform
uniform_buffer_object_index_of_not_active_block,
uniform_buffer_object_inherit_and_override_layouts, and
uniform_buffer_object_repeat_global_scope_layouts to assertion fail.
This is fixed in the next commit.
Signed-off-by: Ian Romanick <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=80d055c1c05fae6cd2e1dc5c8a10e18d992066d2
Author: Ian Romanick <[email protected]>
Date: Tue Jan 22 00:59:56 2013 -0500
glsl: Add link_uniform_blocks to calculate all UBO data at link-time
Calculate all of the block member offsets, the IndexNames, and
everything else to do with every UBO.
Signed-off-by: Ian Romanick <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=1c1517cbd3fcebdae2c09137b094ff6957d259fc
Author: Ian Romanick <[email protected]>
Date: Mon Jan 21 17:33:47 2013 -0500
glsl: Add a visitor to determine whether a uniform block is ever used
Signed-off-by: Ian Romanick <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=aa04b32464731fae2ff5e565b1f2b88b20479282
Author: Ian Romanick <[email protected]>
Date: Tue Jan 22 00:44:58 2013 -0500
glsl: Lower UBO references using link-time data instead of compile-time data
Pretty much all of the compile-time, per-compilation unit block data is
about to get the axe.
Signed-off-by: Ian Romanick <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=0241d3d23131036b550057bd885de50968c2d3a9
Author: Ian Romanick <[email protected]>
Date: Tue Jan 22 00:23:24 2013 -0500
glsl: Add gl_uniform_buffer_variable::IndexName field
glGetUniformIndices requires that the block instance index not be
present in the name of queried uniforms. However,
gl_uniform_buffer_variable::Name will include the instance index. The
IndexName field is added to handle this difference.
Note that currently IndexName will always point to the same string as
Name. This will change soon.
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Carl Worth <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=d78b57623e11e422b6d6138262dc650146144a7b
Author: Ian Romanick <[email protected]>
Date: Tue Jan 22 00:06:10 2013 -0500
glsl: Make the align function available elsewhere in the linker
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Carl Worth <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=d5e12a8fb65c7508fb863da93d5fa80cbbd9b7e4
Author: Ian Romanick <[email protected]>
Date: Mon Jan 21 23:57:45 2013 -0500
glsl: Calculate link-time uniform block data without using compile-time
block data
Pretty much all of the compile-time, per-compilation unit block data is
about to get the axe.
Signed-off-by: Ian Romanick <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=c3dfe86f5051910f9b8e78e963901098a340cb2f
Author: Ian Romanick <[email protected]>
Date: Mon Jan 21 23:43:25 2013 -0500
glsl: Assert that interfaces, like structures, are not seen as leaf types
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Carl Worth <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=a333f0b6bc1349816f0b65d2f1d9b85aa16fdc69
Author: Ian Romanick <[email protected]>
Date: Mon Jan 21 23:42:19 2013 -0500
glsl: Add new uniform_field_visitor::process variant
This flavor takes a type and a base name. It will be used to handle
cases where the block name (instead of the instance name) is used for an
interface block.
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Carl Worth <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=1d4751a52247c07b719dff237b756527c5cb0266
Author: Ian Romanick <[email protected]>
Date: Mon Jan 21 23:27:30 2013 -0500
glsl: Recurse into uniform blocks just like uniform structures
v2: Inspite of the spell checker, spell recurse correctly. Suggested by
Carl Worth.
Signed-off-by: Ian Romanick <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=0992154070bab5230b336dafb46d40a723bb35be
Author: Ian Romanick <[email protected]>
Date: Mon Jan 21 23:06:45 2013 -0500
glsl: Handle instance array declarations
v2: Add a comment and an assertion about the array size in the
non-instance name case. Suggested by Paul Berry.
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Paul Berry <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=273e834f1e81d2904c4a4478bef61e7edb3cc0aa
Author: Ian Romanick <[email protected]>
Date: Mon Jan 21 23:01:33 2013 -0500
glsl: Track blocks in the symbol table using the glsl_type instead of the
gl_uniform_block
Eventually the gl_uniform_block information won't be calculated until
linking. Block names need to be checked for name clashes during
compiling, so we have to track it differently.
v2: Update the commit message. Suggested by Carl Worth.
Signed-off-by: Ian Romanick <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=6095a8db55bcbf147c045b9b0c040f999c62e7b9
Author: Ian Romanick <[email protected]>
Date: Mon Jan 21 22:49:39 2013 -0500
glsl: Add new uniform_field_visitor::visit_field variant
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Carl Worth <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=51ce8fc22516fa8ffcc97535e1803a97364f4eeb
Author: Ian Romanick <[email protected]>
Date: Mon Jan 21 22:41:13 2013 -0500
glsl: Modify uniform_field_visitor::visit_field to take a row_major
parameter
Not used yet, but the UBO layout visitor will use this.
v2: Remove a spruious hunk. This is moved to the patch "glsl: Remove
ir_variable::uniform_block". Suggested by Carl Worth.
Signed-off-by: Ian Romanick <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=597d9513478fefc201789d9cb13f2b17b9e236d1
Author: Ian Romanick <[email protected]>
Date: Mon Jan 21 22:32:07 2013 -0500
glsl: Modify uniform_field_visitor::recursion to take a row_major parameter
Not used yet, but the UBO layout visitor will use this.
v2: Add some commentary as to why row_major is always set to false in
process. Suggesed by Paul Berry.
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Carl Worth <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=c8b6ff4e0680e93bd69e6f14bc00d7a5493ea139
Author: Ian Romanick <[email protected]>
Date: Mon Jan 21 22:18:16 2013 -0500
glsl: Add a predicate to determine whether a variable is an interface block
For the first declaration below, there will be an ir_variable named
"instance" whose type and whose instance_type will be the same
glsl_type. For the second declaration, there will be an ir_variable
named "f" whose type is float and whose instance_type is B2.
"instance" is an interface instance variable, but "f" is not.
uniform B1 {
float f;
} instance;
uniform B2 {
float f;
};
v2: Copy the comment message documentation into the code. Suggested by
Paul Berry.
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Carl Worth <[email protected]>
Reviewed-by: Paul Berry <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=09f1331f2ccfc13617e709ba767b7a36e0a45880
Author: Ian Romanick <[email protected]>
Date: Mon Jan 21 22:07:34 2013 -0500
glsl: Require that indices into uniform block arrays be constants
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Carl Worth <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=2e6f75eb3ed430822f2bb2fe33bde6689db8634d
Author: Ian Romanick <[email protected]>
Date: Mon Jan 21 21:51:15 2013 -0500
glsl: Add ir_variable::interface_type field
For variables that are in an interface block or are an instance of an
interface block, this is the GLSL_TYPE_INTERFACE type for that block.
Convert the ir_variable::is_in_uniform_block method added in the
previous commit to use this field instead of ir_variable::uniform_block.
v2: Fix the place-holder comment on ir_variable::interface_type.
Suggested by Paul Berry.
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Carl Worth <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=4ead17e955b01208969506c69c60e88cff5605a0
Author: Ian Romanick <[email protected]>
Date: Fri Dec 14 12:00:14 2012 -0800
glsl: Add ir_variable::is_in_uniform_block predicate
The way a variable is tested for this property is about to change, and
this makes the code easier to modify.
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Carl Worth <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=902ee70db4218568c679c025145125e1d4c92cda
Author: Ian Romanick <[email protected]>
Date: Tue Dec 11 12:14:03 2012 -0800
glsl: Generate an interface type for uniform blocks
If the block has an instance name, add the instance name to the symbol
table instead of the individual fields.
Fixes the piglit test interface-name-access-without-interface-name.vert
for real.
Signed-off-by: Ian Romanick <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=33730446563b32e878bba1b32bbf35432fa7a675
Author: Ian Romanick <[email protected]>
Date: Tue Dec 11 12:11:16 2012 -0800
glsl: Add GLSL_TYPE_INTERFACE
Interfaces are structurally identical to structures from the compiler's
point of view. They have some additional restrictions, and generally
GPUs use different instructions to access them. Using a different base
type should make this a bit easier.
This commit also adds the glsl_type::interface_packing fields. For
GLSL_TYPE_INTERFACE types, this will track the specified packing mode.
It is analogous to gl_uniform_buffer::_Packing.
v2: Add serveral missing GLSL_TYPE_INTERFACE cases in switch-statements.
v3: Add information about glsl_type::interface_packing. Move row_major
checking in glsl_type::record_key_compare from this patch to the
previous patch. Both suggested by Paul Berry.
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Paul Berry <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=4628ddba2736e3e3d72e928feaa38713fa62e02a
Author: Ian Romanick <[email protected]>
Date: Tue Dec 11 12:09:32 2012 -0800
glsl: Add row_major field to glsl_struct_field
For now, this will always be false. In the near future, an "interface"
type will be added that shares a lot of infrastructure with structures.
v2: Move row_major checking in glsl_type::record_key_compare from the
next patch to this patch. Suggested by Paul Berry.
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Carl Worth <[email protected]>
Reviewed-by: Paul Berry <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=953e5a8a9fc9641b94de4163691c7d6356002323
Author: Ian Romanick <[email protected]>
Date: Sat Dec 8 17:38:30 2012 -0800
glsl: Refactor out processing of structure fields
This will soon also be used for processing interface block fields.
v2: Add a comment explaining the interface of
ast_process_structure_or_interface_block. Suggested by Paul Berry.
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Carl Worth <[email protected]>
Reviewed-by: Paul Berry <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=3f5721cbf726cd70ac1bf55ba0aafb6b5078a58f
Author: Ian Romanick <[email protected]>
Date: Tue Dec 11 12:13:30 2012 -0800
glsl: Parse interface array size
The size is parsed and stored in the AST, but it is not used yet.
Processing of the array size is added in the patch "glsl: Handle
instance array declarations"
v2: Update the commit message (suggested by Carl Worth). Add a comment
to ast_uniform_block::array_size (suggested by Paul Berry).
Signed-off-by: Ian Romanick <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=bfba91b62732e34e3afe6b84975c94014671105d
Author: Kenneth Graunke <[email protected]>
Date: Tue Nov 27 23:45:17 2012 -0800
glsl: Parse non-array uniform block instance names in GLSL ES 3.00.
In GLSL ES 3.00 (and GLSL 1.50), uniform blocks can have an associated
"instance name", which essentially namespaces the variables inside.
This patch adds basic parsing for this new feature, but doesn't yet hook
it up to actually do anything yet.
It does not support for arrays of interface blocks; a later commit will
take care of that.
This change temporarily regresses the piglit test
interface-name-access-without-interface-name.vert. This shader failed
to compile before (the expected result), but it failed to compile for
the wrong reason. This is not a real regression.
v2: Add some comments to ast_uniform_block::instance_name. Suggested by
Paul Berry.
Reviewed-by: Carl Worth <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=6f661e5db8b0ff773548d6d296eca2d93c4c25f9
Author: Kenneth Graunke <[email protected]>
Date: Tue Nov 27 23:21:56 2012 -0800
glsl: Refactor uniform block parser rules.
The existing code has a lot of duplication; the only difference between
the two cases is whether we merge in an additional layout qualifier.
Apparently creating a layout_qualifieropt rule that can be empty causes
a lot of conflicts and confusion. However, refactoring out the guts of
the ast_uniform_block creation works fine.
Reviewed-by: Carl Worth <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=9d630ff1ef4789d53c8d8ff1f0866110bbb4968e
Author: Ian Romanick <[email protected]>
Date: Thu Dec 13 02:25:34 2012 -0800
linker: Refactor intra-stage block compatabililty testing
Also slightly change the compatibility test. Instead of comparing the
offsets of the block variables, compare the packing mode of the blocks.
Ideally we don't want to assign the offsets until a later stage of
linking.
This is put in a new file called link_uniform_blocks.cpp. Some new
functions related to uniform blocks are going to live in that file as
well.
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Carl Worth <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=091d330609ab9d3a2bea46fc1b58094461ecedae
Author: Ian Romanick <[email protected]>
Date: Thu Dec 13 02:13:30 2012 -0800
mesa: Track the packing mode of a UBO in gl_uniform_buffer
This allows the next patch to verify that two uniform blocks match
without first calculating the locations of the fields.
Signed-off-by: Ian Romanick <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=c3fbb57910bbe87f2a415e1e3e9f9de80909b4fe
Author: Ian Romanick <[email protected]>
Date: Tue Dec 11 12:56:03 2012 -0800
glsl: Replace most default cases in switches on GLSL type
This makes it easier to find switch-statements that need to be updated
after a new GLSL_TYPE_* is added because the compiler will generate a
warning.
Switch-statements that only had a small number of cases (e.g.,
everything in ir_constant_expression.cpp) were not modified. I may
regret that decision when we eventually add support for doubles.
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Carl Worth <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=46af26d90ff6d87df47c60d47b3936e5df0b05a0
Author: Eric Anholt <[email protected]>
Date: Wed Jan 23 14:30:05 2013 -0800
i965: Correct gen6+ guardband calculation.
Too much attention was paid to the first paragraphs, and not enough to
the last little note that "oh, by the way, the rendered things
themselves still have to be clipped to just 8192 wide/high".
Fixes GTF's clip.c test with 4096 or higher width on ivb, where one of
the triangles got the upper half of its pixels dropped.
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=b99cf0fefb26a81619ba19bd55d68227ec86b910
Author: Kenneth Graunke <[email protected]>
Date: Wed Jan 23 15:27:39 2013 -0800
i965: Use GL_RED for DEPTH_TEXTURE_MODE in ES 3.0 for unsized formats.
Khronos has apparently decided that depth textures with sized formats
(allowed with ARB_internalformat_query or ES 3.0) should be treated as
GL_RED, while unsized formats (an existing feature) should be treated
as GL_INTENSITY for compatibility with ES 2.0.
Ian is proposing changes to ARB_internalformat_query which will make
this actually legal and consistent.
A similar problem exists with GL 4.2, but we're going to ignore that
for the time being.
Tested on Ivybridge: no Piglit regressions; fixes 4 es3conform tests:
- depth_texture_fbo
- depth_texture_fbo_clear
- depth_texture_teximage
- depth_texture_texsubimage
Reviewed-by: Ian Romanick <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=c9f73fb0877aee03e718578555a1fd8b225bfdfc
Author: Chad Versace <[email protected]>
Date: Wed Nov 21 09:28:36 2012 -0800
i965: Bump maximum supported ES2 context version to 3.0
Since patch "i965: Validate requested GLES context version in
brwCreateContext", we have been able to create ES 3.0 contexts due to the
max version check. So...bump the max version.
Reviewed-by: Kenneth Graunke <[email protected]>
Reviewed-by: Ian Romanick <[email protected]>
Signed-off-by: Chad Versace <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=088a6415ff200465d40cb829065439de4d17147a
Author: Paul Berry <[email protected]>
Date: Tue Aug 7 12:39:30 2012 -0700
i965/Gen6+: Enable ARB_ES3_compatibility extension
Reviewed-by: Ian Romanick <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=3ca94a1451a1ce9054883dc2170c4821652b1519
Author: Ian Romanick <[email protected]>
Date: Wed Aug 8 15:25:00 2012 -0700
mesa/es3: Enable ES 3.0 API and shading language version
v2: Add ARB_internalformat_query to the list of required extensions.
v3: Add OES_depth_texture_cube_map to the list of required extensions.
Signed-off-by: Ian Romanick <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=5bd86b26df30d32412ca1b7ac02892ab5e10c6ef
Author: Ian Romanick <[email protected]>
Date: Tue Aug 21 15:33:04 2012 -0700
intel: Enable S3TC extensions always
Always enable the use of pre-compressed texture data. The ability to
perform on-line compression still requires the presence of libtxc_dxtn
or an explicit driconf over-ride. Applications that just want to submit
precompessed data when an on-line compressor is not available can look
for the GL_EXT_texture_compression_dxt1 and
GL_ANGLE_texture_compression_dxt[35] extensions.
v2: Only enable the extensions that do not require on-line compression
by default. The previous statement "This should not impact many (if
any) real applications." proved to be false for at least Sauerbraten.
This application mostly submits pre-compressed data, but it also can
submit uncompressed data that it asks the driver to compress.
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Jordan Justen <[email protected]> [v1]
Reviewed-by: Kenneth Graunke <[email protected]> [v1]
Acked-by: Eric Anholt <[email protected]> [v1]
Acked-by: Lee Salzman <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=53f8251107d8a23892f0248db5a12098dadea38f
Author: Ian Romanick <[email protected]>
Date: Sat Dec 1 12:10:08 2012 -0800
mesa: Like EXT_texture_compression_dxt1, advertise
ANGLE_texture_compression_dxt in all APIs
This is technically outside the ANGLE spec, but it seems unlikely to
cause any harm.
v2: Simplify the extension checks by assuming the ANGLE extension will
always be enabled by any driver that enables the EXT. Suggested by
Eric Anholt.
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Eric Anholt <[email protected]>
Acked-by: Lee Salzman <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=d45c6c817df5c00ec12d46bd07fc7735e2c2cd5a
Author: Ian Romanick <[email protected]>
Date: Tue Jan 22 04:24:24 2013 -0500
mesa: Simplify _mesa_choose_tex_format handling of compressed formats
For non-generic compressed format we assert two things:
1. The format has already been validated against the set of available
extensions.
2. The driver only enables the extension if it supports all of the
formats that are part of that extension.
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Eric Anholt <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=a021881ccdaf1d81fb7c8bef4b58b4c037dc4f44
Author: Ian Romanick <[email protected]>
Date: Sat Dec 1 12:02:04 2012 -0800
mesa: Use a single flag for the S3TC extensions that don't require on-line
compression
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Eric Anholt <[email protected]>
Acked-by: Lee Salzman <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=8059c2ea9011110df3632e28a4311de9373b9b4e
Author: Carl Worth <[email protected]>
Date: Tue Jan 22 13:29:01 2013 -0800
i965: Use swizzles to force R, G, and B to 0.0 for ALPHA textures.
Similar to the previous commit, we may be using a texture with actual RGBA
storage for the GL_ALPHA format, so force the color values to 0.0.
This commit fixes the following piglit (sub) tests:
EXT_texture_snorm/fbo-blending-formats
GL_ALPHA16_SNORM
GL_ALPHA8_SNORM
GL_ALPHA_SNORM
Note: Haswell bypasses this swizzle code, so may require an independent fix
for this bug.
Reviewed-by: Eric Anholt <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=33599433c75c0ab433615d08ed7e4002146ba462
Author: Carl Worth <[email protected]>
Date: Tue Jan 22 13:25:37 2013 -0800
i965: Use swizzles to force alpha to 1.0 for RED, RG, or RGB textures.
We may be using a texture with actual RGBA storage for these formats, so
force
the alpha value read to 1.0.
This commit fixes the following piglit (sub) tests:
ARB_texture_float/fb-blending-formats
GL_RGB16F_ARB
EXT_framebuffer_object/fbo-blending-formats
GL_RGB10
GL_RGB12
GL_RGB16
EXT_texture_snorm/fbo-blending-formats
GL_RGB16_SNORM
GL_RGB8_SNORM
GL_RGB_SNORM
These test improvements depend on the previous commit as well. That commit
smashes alpha to 1.0 for the case of ReadPixels (so fixes "FBO testing" as
reported by this test), while this commit smashes alpha to 1.0 for the case
of
texturing (fixed the "window testing" as reported by this test).
Note: Haswell bypasses this swizzle code, so may require an independent fix
for this bug.
Reviewed-by: Eric Anholt <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=570ed2be7d776211e1ca2a7a4c44ee6a1d141714
Author: Carl Worth <[email protected]>
Date: Mon Jan 21 12:16:27 2013 -0800
ReadPixels: Force ALPHA to 1 while rebasing RGBA values for GL_RGB format
When performing a ReadPixels operation, we may be reading from a buffer that
stores alpha values, but that is actually representing a buffer with no
alpha
channel. In this case, while rebasing the values, touch up all alpha values
read to 1.0.
This commit fixes the following piglit (sub) tests:
ARB_texture_float/fbo-colormask-formats
GL_RBG16F_ARB
EXT_texture_snorm/fbo-colormask-formats
GL_RGB16_SNORM
GL_RGB8_SNORM
GL_RGB_SNORM
It likely improves the results of other tests as well, but a PASS remains
elusive due to additional bugs.
Reviewed-by: Brian Paul <[email protected]>
Reviewed-by: Anuj Phogat <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=b961ba44ed01d4aa590609fc91ab1250ac7d9559
Author: Carl Worth <[email protected]>
Date: Mon Jan 21 12:18:02 2013 -0800
i965: Examine _BaseFormat when deciding to perform xRGB_alpha fixups
The renderbuffer's Format field may have an alpha channel even when the
underlying _BaseFormat does not. This can happen when mesa chooses to use
RGBA16 for an RGB16 format, for example.
So look at _BaseFormat when deciding whether to fixup the blend factors.
This test improves the results of at least the following piglit tests:
EXT_frambebuffer_object/fbo-blending-formats
{GL_RGB10, GL_RGB12, GL_RGB16}
EXT_texture_snorm/fbo-blending-formats
{GL_RGB16_SNORM, GLRGB8_SNORM, GL_RGB_SNORM}
But none of these actually change from FAIL to PASS yet. The R, G, and B
probe
values are fixed with this commit, but the tests still fail because the
alpha
values are still wrong.
Reviewed-by: Eric Anholt <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=06424376065a4a1e44920a29497936691d88598f
Author: José Fonseca <[email protected]>
Date: Tue Jan 22 20:54:17 2013 +0000
scons: Fix source lists parsing on Windows.
/ vs \ mismatch was causing .objs to be put in the source tree, causing
breakeage when doing different build types in the same tree (eg., debug
vs release).
Fix this by normalizing everything to / slashes.
It's probably a good idea to purge all .objs from source tree to prevent
issues completely.
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=60315e3eaf54a7eb646b787daa5f3ae560aee7a6
Author: Matt Turner <[email protected]>
Date: Tue Jan 22 21:34:25 2013 -0800
GL3.txt: i965 supports ARB_base_instance
Added in commit cdd3f549.
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=bd8045d4c516b9c753d00dcc4b962a3e634ad506
Author: Brian Paul <[email protected]>
Date: Tue Jan 22 16:42:54 2013 -0700
wmesa: include api_exec.h to fix compilation
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=26a05b5005fd89cad0864258a16951d812b923f8
Author: Brian Paul <[email protected]>
Date: Tue Jan 22 16:41:56 2013 -0700
draw: fix MSVC divide-by-zero compilation error
Kind of lame, but it works.
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=cdd3f5496ad2d6a252bd7ad78f5034db549d4921
Author: Kenneth Graunke <[email protected]>
Date: Fri Jan 18 11:11:10 2013 -0800
i965: Implement the GL_ARB_base_instance extension.
Thanks to Fredrik Höglund, all the hard work was already done.
Tested using a modified oglconform (that actually runs these tests on
our driver); it looks like there may be some bugs when using client
arrays. All applicable non-compatibility tests passed.
For now, only enable it in core profiles.
Reviewed-by: Eric Anholt <[email protected]>
Tested-by: Ian Romanick <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=0d108116bd80b757fb01a84a9f1946ef870b57b8
Author: Matt Turner <[email protected]>
Date: Thu Jan 17 23:51:04 2013 -0800
glsl/build: Build libglcpp and libglslcore in builtin_compiler
And reuse them if not cross compiling.
Tested-by: Andreas Boll <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=952e6e9f3b0eb179f67345f00e5a7f1dbaa7bdd5
Author: Matt Turner <[email protected]>
Date: Thu Jan 17 23:28:42 2013 -0800
glsl/Makefile.sources: Correct BUILTIN_COMPILER_CXX_FILES
Squashed with two reverts:
Revert "android: Update for builtin_stubs.cpp move"
This reverts commit c0def90ede1e939173041b8785303de90f8fdc6c.
Revert "scons: Update for builtin_stubs.cpp"
This reverts commit 8ac4b82699ad0a59ae6ae6d3415702eaa5d4fe3b.
Tested-by: Andreas Boll <[email protected]>
Tested-on-Android-by: Chad Versace <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=2a710543968b6710368d53dbff350d2d23e08268
Author: Matt Turner <[email protected]>
Date: Mon Oct 1 13:28:51 2012 -0700
build: Use AX_PROG_FLEX
Tested-by: Andreas Boll <[email protected]>
Reviewed-by: Eric Anholt <[email protected]>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=47248
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=b68b85224d38fe9c1f6e10af338e1ff2519e689b
Author: Matt Turner <[email protected]>
Date: Mon Oct 1 13:11:30 2012 -0700
build: Use AX_PROG_BISON
No one tests yacc/byacc. Let's just request bison specifically.
Tested-by: Andreas Boll <[email protected]>
Reviewed-by: Eric Anholt <[email protected]>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=46815
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=3791ce05eb3c169c55b1d6eb5e613684e2cec4df
Author: Matt Turner <[email protected]>
Date: Thu Jan 17 14:40:21 2013 -0800
builtin_compiler/build: Use generated parser files
... instead of generating them again.
Tested-by: Andreas Boll <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=efd201caa5cbfaf10d0ecad219046a9fe3593c4b
Author: Matt Turner <[email protected]>
Date: Tue Jan 15 11:45:40 2013 -0800
glsl/build: Build tests via the glsl Makefile
Tested-by: Andreas Boll <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=86d30dea3c13d29ef8d39bc18db63a0441051975
Author: Matt Turner <[email protected]>
Date: Mon Jan 14 22:00:08 2013 -0800
glsl/build: Build glcpp via the glsl Makefile
Removing the subdirectory recursion provides a small speed up.
Tested-by: Andreas Boll <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=cc9f609cb9175e727b91f8dffa5800818be582da
Author: Matt Turner <[email protected]>
Date: Mon Jan 14 16:16:00 2013 -0800
glsl/build: Don't build builtin_compiler separately if not cross compiling
Reduces the number of times that src/glsl/ is compiled when not cross
compiling.
Tested-by: Andreas Boll <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=569f0e400aca0e7bfd13374384fb8d92caf869bc
Author: Matt Turner <[email protected]>
Date: Mon Jan 14 14:24:17 2013 -0800
glsl/build: Don't build glsl_compiler
Use glslparsertest from piglit instead.
Tested-by: Andreas Boll <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=ab74fee5e1a3fc3323b7238278637b232c2d0d95
Author: Brian Paul <[email protected]>
Date: Mon Jan 21 11:41:02 2013 -0700
draw: fix problem in screen-space interpolation clip code
I don't see how this could have ever worked right.
The screen-space interpolation code uses the vertex->data[pos_attr]
position which contain window coords. But window coords are only
computed for the unclipped vertices; the clipped vertices have
undefined window coords (see draw_cliptest_tmp.h).
Use the vertex clip coords instead which are always defined.
Fixes http://bugs.freedesktop.org/show_bug.cgi?id=55476
(piglit fbo-blit-stretch failure on softpipe)
Note: This is a candidate for the 9.0 branch.
Reviewed-by: José Fonseca <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=ed643d6b2fa11082f617f24040c73d2c58ca09bf
Author: Brian Paul <[email protected]>
Date: Mon Jan 21 11:37:07 2013 -0700
draw: improve the clipper debug/printf code
Reviewed-by: José Fonseca <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=4a938ef7136a89c828ebb16effe1bc5bea08b7d7
Author: Brian Paul <[email protected]>
Date: Mon Jan 21 11:32:49 2013 -0700
draw: add new debug code and comments in clip code template
In debug builds, set clipped vertex window coordinates to NaN values
to help debugging. Otherwise, we're just leaving the coordinate in clip
space and it's invalid to use it later expecting it to be a window coord.
Reviewed-by: José Fonseca <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=547a418888c455a3f5158c147a155292f398ea34
Author: Brian Paul <[email protected]>
Date: Mon Jan 21 08:59:25 2013 -0700
swrast: fix blit code's nearest/linear coordinate arithmetic
Fixes piglit's fbo-blit-stretch test.
Reviewed-by: José Fonseca <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=b70b4862491121d5cc4444cea34f2f86a3aa5ded
Author: Brian Paul <[email protected]>
Date: Mon Jan 21 08:59:25 2013 -0700
swrast: fix incorrect width for direct/nearest blit
Reviewed-by: José Fonseca <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=728bf86a23f6de137c0871ea87b09e75e55468a9
Author: Brian Paul <[email protected]>
Date: Mon Jan 21 08:59:25 2013 -0700
swrast: move resampleRow setup code in blit_nearest()
The resampleRow setup depends on pixelSize. For color buffers,
we don't know the pixelSize until we're in the buffer loop. Move
that code inside the loop.
Fixes: http://bugs.freedesktop.org/show_bug.cgi?id=59541
Reviewed-by: José Fonseca <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=0a60ea4ddce3d7e0f07f9a8a94cd712c26401911
Author: Andreas Boll <[email protected]>
Date: Tue Jan 22 21:21:09 2013 +0100
docs: import release notes for 9.0.2, add news item
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=9a0973044ed097a6b33b9e1c7c74045fd1affa19
Author: José Fonseca <[email protected]>
Date: Tue Mar 8 13:18:56 2011 +0000
scons: Disable frame pointer omission for all build types except release.
In particular for checked builds, where debug_backtrace_capture relies
on it.
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=de0057caa66207c5eb94bd251ab357309f78b60c
Author: José Fonseca <[email protected]>
Date: Tue Jan 22 18:35:13 2013 +0000
nouveau/build: Fix build failures when drm is not in /usr/include.
Fixes failures to include libdrm/nouveau.h when drm is not installed in
/usr/include.
Reviewed-by: Matt Turner <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=a56dfd99e2cf2b7a0916cb59d2866303a627b02f
Author: Michel Dänzer <[email protected]>
Date: Mon Jan 21 19:08:24 2013 +0100
radeon/llvm: Handle LP_CHAN_ALL in emit_fetch_immediate().
Fixes piglit spec/ARB_sampler_objects/sampler-incomplete and
spec/EXT_texture_swizzle/depth_texture_mode_and_swizzle.
Reviewed-by: Tom Stellard <[email protected]>
Signed-off-by: Michel Dänzer <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=121d19de920212225586c9269f2d34ab7e6e1aec
Author: Kenneth Graunke <[email protected]>
Date: Mon Jan 21 10:42:50 2013 -0800
build: Fix build on systems where /usr/bin/python isn't python 2.
configure.ac sets up a PYTHON2 variable, which is what we want
AX_PYTHON_MODULE to use (since we only use Python 2 for now).
NOTE: This is a candidate for the 9.0 branch.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=31598
Reviewed-by: Matt Turner <[email protected]>
Reviewed-by: Andreas Boll <[email protected]>
_______________________________________________
mesa-commit mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/mesa-commit
