- Dec 15, 2015
-
-
Emil Velikov authored
Signed-off-by:
Emil Velikov <emil.velikov@collabora.com>
-
- Dec 14, 2015
-
-
Emil Velikov authored
Signed-off-by:
Emil Velikov <emil.velikov@collabora.com>
-
- Dec 12, 2015
-
-
Fixes arb_shader_image_load_store/execution/load-from-cleared-image.shader_test. Couldn't reproduce any significant FPS regression in CPU-bound benchmarks from the Finnish benchmarking system on neither VLV nor BSW after 30 runs with 95% confidence level. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=92849 Cc: Chris Wilson <chris@chris-wilson.co.uk> Cc: Jason Ekstrand <jason.ekstrand@intel.com> Cc: "11.0 11.1" <mesa-stable@lists.freedesktop.org> Tested-by:
Jordan Justen <jordan.l.justen@intel.com> Reviewed-by:
Kristian Høgsberg <krh@bitplanet.net> (cherry picked from commit 595c8180)
-
This adds the double code to the geometry shader input handling. Reviewed-by:
Michel Dänzer <michel.daenzer@amd.com> Cc: "11.0 11.1" <mesa-stable@lists.freedesktop.org> Signed-off-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit e307cfa7)
-
This handles loading doubles from LDS properly. Reviewed-by:
Michel Dänzer <michel.daenzer@amd.com> Cc: "11.0 11.1" <mesa-stable@lists.fedoraproject.org> Signed-off-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit 8c9e40ac)
-
This fixes: glsl-1.50/execution/geometry/dynamic_input_array_index.shader_test my profanity. We need to load the AR register with the value from the index reg Cc: "11.0 11.1" <mesa-stable@lists.freedesktop.org> Signed-off-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit cce38640)
-
This fixes: gs-input-array-vec4-index-rd The others run out of gprs unfortunately. Cc: "11.0 11.1" <mesa-stable@lists.freedesktop.org> Signed-off-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit 38542921)
-
These utilities are to be used to do things like integer adds and multiplies to be used in calculating the LDS offsets etc. It handles CAYMAN MULLO differences as well. Signed-off-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit 0696ebc8)
-
This will be used in the tess shaders. Reviewed-by:
Oded Gabbay <oded.gabbay@gmail.com> Signed-off-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit 4d64459a)
-
This fixes: arb_transform_feedback3-ext_interleaved_two_bufs_gs arb_transform_feedback3-ext_interleaved_two_bufs_gs_max transform-feedback-builtins If we are only emitting one ring, then emit all output buffers on it. Cc: "11.0 11.1" <mesa-stable@lists.freedesktop.org> Signed-off-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit e97ac006) [Emil Velikov: squash trivial conflicts] Signed-off-by:
Emil Velikov <emil.l.velikov@gmail.com> Conflicts: src/gallium/drivers/r600/r600_shader.c
-
The only effect here is a space savings - 822 programs in shader-db affected with the following overall change: total bytes used in shared programs : 44154976 -> 44139880 (-0.03%) Fixes: 641eda0c (nv50/ir: r63 is only 0 if we are using less than 63 registers) Signed-off-by:
Ilia Mirkin <imirkin@alum.mit.edu> Cc: "11.0 11.1" <mesa-stable@lists.freedesktop.org> (cherry picked from commit f920f8eb)
-
... and allow the "binding" qualifier in ES 3.1 as well. GLSL ES 3.1 incorporates only a few features from the extension ARB_shading_language_420pack: the relaxed qualifier ordering requirements and the binding qualifier. Cc: "11.1" <mesa-stable@lists.freedesktop.org> Reviewed-by:
Kenneth Graunke <kenneth@whitecape.org> (cherry picked from commit eca846e7)
-
These features would not have been enabled with #version 420 otherwise. Cc: "11.1" <mesa-stable@lists.freedesktop.org> Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> (cherry picked from commit 79da7220)
-
This interaction was missed in the addition of ARB_image_load_store. Cc: "11.0 11.1" <mesa-stable@lists.freedesktop.org> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=93266 Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Reviewed-by:
Ian Romanick <ian.d.romanick@intel.com> (cherry picked from commit c200e606)
-
The one and only place where the FS backend allows reladdr is on uniforms. For locals, inputs, and outputs, we lower it away before the backend ever sees it. This commit gets rid of the dead indirect handling code. Cc: "11.0" <mesa-stable@lists.freedesktop.org> Reviewed-by:
Kenneth Graunke <kenneth@whitecape.org> (cherry picked from commit 22c273de)
-
Cc: "11.0" <mesa-stable@lists.freedesktop.org> Reviewed-by:
Kenneth Graunke <kenneth@whitecape.org> (cherry picked from commit abb569ca)
-
Cc: "11.0" <mesa-stable@lists.freedesktop.org> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=92909 Reviewed-by:
Kenneth Graunke <kenneth@whitecape.org> (cherry picked from commit 05bdc21f)
-
Cc: "11.0" <mesa-stable@lists.freedesktop.org> Reviewed-by:
Kenneth Graunke <kenneth@whitecape.org> (cherry picked from commit 13ad8d03)
-
Previously, the VS_OPCODE_PULL_CONSTANT_LOAD opcode operated on vec4-aligned byte offsets on Iron Lake and below and worked in terms of vec4 offsets on Sandy Bridge. On Ivy Bridge, we add a new *LOAD_GEN7 variant which works in terms of vec4s. We're about to change the GEN7 version to work in terms of bytes, so this is a nice unification. Cc: "11.0" <mesa-stable@lists.freedesktop.org> Reviewed-by:
Kenneth Graunke <kenneth@whitecape.org> (cherry picked from commit e3e70698)
-
Cc: "11.1" <mesa-stable@lists.freedesktop.org> Reviewed-by:
Marek Olšák <marek.olsak@amd.com> (cherry picked from commit d5a5dbd7)
-
According to nvdisasm both the immediate and non-imm cases use the same bits. Both of these flags are quite rarely set though. Signed-off-by:
Ilia Mirkin <imirkin@alum.mit.edu> Cc: "11.0 11.1" <mesa-stable@lists.freedesktop.org> (cherry picked from commit 1d708aac)
-
We actually leave the sampler unset for OP_TXF, which caused the GK104+ logic to treat some texel fetches as indirect. While this works, it's incredibly wasteful. This only happened when the texture was > 0 (since sampler remained == 0). Signed-off-by:
Ilia Mirkin <imirkin@alum.mit.edu> Cc: "11.0 11.1" <mesa-stable@lists.freedesktop.org> (cherry picked from commit 63b85040)
-
Cc: 11.1 <mesa-stable@lists.freedesktop.org> Reviewed-by:
Alex Deucher <alexander.deucher@amd.com> (cherry picked from commit 32f05fad)
-
The workarounds are too hacky to enable them by default and otherwise MPEG4 doesn't work reliably. v2: add docs/envvars.html, CC stable and fix typos Signed-off-by:
Christian König <christian.koenig@amd.com> Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com> (v1) Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu> (v1) Cc: "11.1.0" <mesa-stable@lists.freedesktop.org> (cherry picked from commit a2c5200a)
-
The elemental demo hits this case. Signed-off-by:
Ilia Mirkin <imirkin@alum.mit.edu> Cc: "11.0 11.1" <mesa-stable@lists.freedesktop.org> (cherry picked from commit db072d20)
-
This allows validation to be done on rendering calls also. Fixes 3 dEQP-GLES31.functional.separate tests. Cc: "11.1" <mesa-stable@lists.freedesktop.org> Reviewed-by:
Tapani Pälli <tapani.palli@intel.com> Cc: Kenneth Graunke <kenneth@whitecape.org> (cherry picked from commit 4dd096d7)
-
From Section 11.1.3.11 (Validation) of the GLES 3.1 spec: "An INVALID_OPERATION error is generated by any command that trans- fers vertices to the GL or launches compute work if the current set of active program objects cannot be executed, for reasons including:" It then goes on to list the rules we validate in the _mesa_validate_program_pipeline() function. For ValidateProgramPipeline the only mention of generating an error is: "An INVALID_OPERATION error is generated if pipeline is not a name re- turned from a previous call to GenProgramPipelines or if such a name has since been deleted by DeleteProgramPipelines," Which we handle separately. This fixes: ES31-CTS.sepshaderobjs.PipelineApi No regressions on the eEQP 3.1 tests. Cc: Gregory Hainaut <gregory.hainaut@gmail.com> Reviewed-by:
Tapani Pälli <tapani.palli@intel.com> (cherry picked from commit c3ec12ec) Nominated-by:
Emil Velikov <emil.velikov@collabora.com>
-
Cc: "11.1" <mesa-stable@lists.freedesktop.org> Reviewed-by:
Tapani Pälli <tapani.palli@intel.com> Cc: Kenneth Graunke <kenneth@whitecape.org> https://bugs.freedesktop.org/show_bug.cgi?id=93180 (cherry picked from commit da1a0136)
-
This fixes an issue where the addition of the FLAT qualifier in varying_matches::record() can break the expected varying order. It also avoids a future issue with the relaxing of interpolation qualifier matching constraints in GLSL 4.50. V2: (by Timothy Arceri) * reworked comment slightly Signed-off-by:
Gregory Hainaut <gregory.hainaut@gmail.com> Reviewed-by:
Timothy Arceri <timothy.arceri@collabora.com> Reviewed-by:
Tapani Pälli <tapani.palli@intel.com> (cherry picked from commit 2ab9cd0c) Nominated-by:
Timothy Arceri <timothy.arceri@collabora.com>
-
GL_ARB_separate_shader_objects allow matching by name variable or block interface. Input varyings can't be removed because it is will impact the location assignment. This fixes the bug 79783 and likely any application that uses GL_ARB_separate_shader_objects extension. V2 (by Timothy Arceri): * simplify now that builtins are not set as always active Signed-off-by:
Gregory Hainaut <gregory.hainaut@gmail.com> Reviewed-by:
Timothy Arceri <timothy.arceri@collabora.com> Reviewed-by:
Tapani Pälli <tapani.palli@intel.com> https://bugs.freedesktop.org/show_bug.cgi?id=79783 (cherry picked from commit 8117f46f) Nominated-by:
Timothy Arceri <timothy.arceri@collabora.com>
-
The value will be set in separate-shader program when an input/output must remains active. e.g. when deadcode removal isn't allowed because it will create interface location/name-matching mismatch. v3: * Rename the attribute * Use ir_variable directly instead of ir_variable_refcount_visitor * Move the foreach IR code in the linker file v4: * Fix variable name in assert v5 (by Timothy Arceri): * Rename functions and reword comments * Don't set always active on builtins Signed-off-by:
Gregory Hainaut <gregory.hainaut@gmail.com> Reviewed-by:
Timothy Arceri <timothy.arceri@collabora.com> Reviewed-by:
Tapani Pälli <tapani.palli@intel.com> (cherry picked from commit 618612f8) Nominated-by:
Timothy Arceri <timothy.arceri@collabora.com>
-
Cc: Gregory Hainaut <gregory.hainaut@gmail.com> Reviewed-by:
Tapani Pälli <tapani.palli@intel.com> (cherry picked from commit 76c09c17)
-
Tested. (cherry picked from commit bfc14796)
-
This is needed to be able to implement the accepted OES extensions. Cc: "11.0 11.1" <mesa-stable@lists.freedesktop.org> Signed-off-by:
Marta Lofstedt <marta.lofstedt@linux.intel.com> Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> (cherry picked from commit 1d5b88e3)
-
Emil Velikov authored
This reverts commit 79f3aaca. The commit (header update) was not needed for the 11.0 branch as opposed to this one (11.1)
-
Emma Anholt authored
If there were src unpacks, changing to the integer MOV instead of float (for example) would change the unpack operation. (cherry picked from commit e3efc4b0)
-
Emma Anholt authored
The caller isn't going to expect it from a return, so it would probably get misinterpreted. If the caller had an unpack in its reg, that's fine, but don't lose track of it. (cherry picked from commit 2591beef)
-
Emma Anholt authored
(cherry picked from commit b70a2f4d)
-
Emma Anholt authored
I apparently broke this in a late refactor, in such a way that I decided its tests were some of those interminable ones that I should just blacklist from my testing. As a result, the refactors related to it were totally wrong. (cherry picked from commit 53b2523c)
-