- Oct 05, 2015
-
-
Lina Versace authored
There are three types of fast clears: a. fast depth clears b. fast singlesample color clears c. fast multisample color clears Function intel_miptree_is_fast_clear_capable() checks if a miptree supports fast clears of type (b). Rename the function to disambiguate what it does: old: intel_miptree_is_fast_clear_capable new: intel_miptree_supports_non_msrt_fast_clear The functionally accidentally rejected multisampled color surfaces because it thought they were singlesample array surfaces. Fix that by explicitly rejecting surfaces with samples > 1. This fix would have been needed before we enabled layered fast singlesample color clears (introduced in gen8), which we want to do eventually. For now, though, this patch changes no behavior; it just fixes how the driver chooses its behavior. Reviewed-by: Anuj Phogat <anuj.phogat@gmail.com>
-
Lina Versace authored
intel_tiling_supports_non_msrt_mcs() and intel_miptree_is_fast_clear_capable() are not used outside of intel_mipmap_tree.c. Reviewed-by: Anuj Phogat <anuj.phogat@gmail.com>
-
Iago Toral authored
We need a virtual destructor when at least one of the class' methods is virtual. Failure to do so might lead to undefined behavior when destructing derived classes. Fixes the following warning: brw_vec4_gs_visitor.cpp: In function 'const unsigned int* brw::brw_gs_emit(brw_context*, gl_shader_program*, brw_gs_compile*, void*, unsigned int*)': brw_vec4_gs_visitor.cpp:703:11: warning: deleting object of polymorphic class type 'brw::vec4_gs_visitor' which has non-virtual destructor might cause undefined behaviour [-Wdelete-non-virtual-dtor] delete gs; Curro: This shouldn't be causing any actual bugs at the moment because gen6_gs_visitor is the only subclass of vec4_visitor destroyed through a pointer of a base class (vec4_gs_visitor *) and its destructor is basically the same as its parent's. Anyway it seems sensible to change this so it doesn't bite us in the future. Reviewed-by: Francisco Jerez <currojerez@riseup.net>
-
Tapani Pälli authored
Fixes following Piglit test: global-scope-binding-qualifier.frag Signed-off-by: Tapani Pälli <tapani.palli@intel.com> Reviewed-by: Samuel Iglesias Gonsálvez <siglesias@igalia.com>
-
Iago Toral authored
In theory we can't break this assertion since the compiler frontend checks that we don't exceed any of the individual limits, but it does not hurt to be extra safe. Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
-
Iago Toral authored
These share the space with UBO surfaces but we need to make sure we allocate enough space for both sets (12 of each) Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
-
Iago Toral authored
Instead of using hard-coded values. Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
-
Iago Toral authored
Instead of using hard-coded values. Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
-
Matt Turner authored
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
-
Matt Turner authored
The thing you want to do with the output files is diff them, which is made more difficult by line numbers changing. Reviewed-by: Alejandro Piñeiro <apinheiro@igalia.com>
-
Ilia Mirkin authored
It seems like things are either coming in slighly wrong, or perhaps uploaded incorrectly, but either way passing them through the translate module seems to fix everything. Eventually we should figure out what's going wrong and fix it "for real", but this should do for now. Signed-off-by: Ilia Mirkin <imirkin@alum.mit.edu> Cc: mesa-stable@lists.freedesktop.org
-
Ilia Mirkin authored
This puts us in line with what the DDX/DRI2 st are expecting. It also happens to work... no idea why, but seems better to have it work than to ask lots of questions. Signed-off-by: Ilia Mirkin <imirkin@alum.mit.edu> Cc: mesa-stable@lists.freedesktop.org
-
Fixes Gallium based DRI drivers failing to load on big endian hosts because they can't find any matching fbconfigs. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=71789 Signed-off-by: Michel Dänzer <michel.daenzer@amd.com> Reviewed-by: Marek Olšák <marek.olsak@amd.com> Tested-by: Ilia Mirkin <imirkin@alum.mit.edu>
-
- Oct 04, 2015
-
-
Timothy Arceri authored
The uniform will only be of a single type so store the data for opaque types in a single array. Cc: Francisco Jerez <currojerez@riseup.net> Cc: Ilia Mirkin <imirkin@alum.mit.edu>
-
Kenneth Graunke authored
Unfortunately it has to stay in gen6_gs_visitor. Signed-off-by: Kenneth Graunke <kenneth@whitecape.org> Reviewed-by: Matt Turner <mattst88@gmail.com>
-
Kenneth Graunke authored
Signed-off-by: Kenneth Graunke <kenneth@whitecape.org> Reviewed-by: Matt Turner <mattst88@gmail.com>
-
Kenneth Graunke authored
Signed-off-by: Kenneth Graunke <kenneth@whitecape.org> Reviewed-by: Matt Turner <mattst88@gmail.com>
-
Kenneth Graunke authored
Geometry and tessellation shaders process multiple vertices; their inputs are arrays indexed by the vertex number. While GLSL makes this look like a normal array, it can be very different behind the scenes. On Intel hardware, all inputs for a particular vertex are stored together - as if they were grouped into a single struct. This means that consecutive elements of these top-level arrays are not contiguous. In fact, they may sometimes be in completely disjoint memory segments. NIR's existing load_input intrinsics are awkward for this case, as they distill everything down to a single offset. We'd much rather keep the vertex ID separate, but build up an offset as normal beyond that. This patch introduces new nir_intrinsic_load_per_vertex_input intrinsics to handle this case. They work like ordinary load_input intrinsics, but have an extra source (src[0]) which represents the outermost array index. v2: Rebase on earlier refactors. v3: Use ssa defs instead of nir_srcs, rebase on earlier refactors. Signed-off-by: Kenneth Graunke <kenneth@whitecape.org> Reviewed-by: Jason Ekstrand <jason.ekstrand@intel.com>
-
Kenneth Graunke authored
get_io_offset() already walks the dereference chain and discovers whether or not we have an indirect; we can just return that rather than computing it a second time via deref_has_indirect(). This means moving the call a bit earlier. By returning a nir_ssa_def *, we can pass back both an existence flag (via NULL checking the pointer) and the value in one parameter. It also simplifies the code somewhat. nir_lower_samplers works in a similar fashion. Signed-off-by: Kenneth Graunke <kenneth@whitecape.org> Reviewed-by: Jason Ekstrand <jason.ekstrand@intel.com>
-
Timothy Arceri authored
Reviewed-by: Iago Toral Quiroga <itoral@igalia.com>
-
- Oct 03, 2015
-
-
Marek Olšák authored
Now st/mesa won't generate 2 variants for this state. Reviewed-by: Michel Dänzer <michel.daenzer@amd.com>
-
Marek Olšák authored
Reviewed-by: Michel Dänzer <michel.daenzer@amd.com>
-
Marek Olšák authored
Reviewed-by: Michel Dänzer <michel.daenzer@amd.com>
-
Marek Olšák authored
This will be a derived state used for changing center->sample and centroid->sample at runtime. Reviewed-by: Michel Dänzer <michel.daenzer@amd.com>
-
Marek Olšák authored
Reviewed-by: Michel Dänzer <michel.daenzer@amd.com>
-
Marek Olšák authored
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
-
Marek Olšák authored
This is only a half of the work. The next patch will handle gl_SampleID/SamplePos, which is the other half of ARB_sample_shading. Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
-
Marek Olšák authored
Required by ARB_sample_shading for drivers that don't want a shader variant in st/mesa. Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu> Acked-by: Roland Scheidegger <sroland@vmware.com>
-
Marek Olšák authored
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
-
Marek Olšák authored
Nothing sets it. Reviewed-by: Brian Paul <brianp@vmware.com> Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
-
Marek Olšák authored
Nothing overrides it. Reviewed-by: Brian Paul <brianp@vmware.com> Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
-
Marek Olšák authored
Nothing overrides it. Reviewed-by: Brian Paul <brianp@vmware.com>
-
Marek Olšák authored
Nothing overrides it. Reviewed-by: Brian Paul <brianp@vmware.com>
-
Marek Olšák authored
Nothing overrides it. Reviewed-by: Brian Paul <brianp@vmware.com>
-
Marek Olšák authored
Nothing overrides it. Reviewed-by: Brian Paul <brianp@vmware.com>
-
Marek Olšák authored
Nothing overrides it. Reviewed-by: Brian Paul <brianp@vmware.com>
-
Marek Olšák authored
Nothing overrides it. Reviewed-by: Brian Paul <brianp@vmware.com>
-
Marek Olšák authored
Nothing overrides it. Reviewed-by: Brian Paul <brianp@vmware.com>
-
Marek Olšák authored
Nothing overrides it. Reviewed-by: Brian Paul <brianp@vmware.com>
-
Marek Olšák authored
Nothing sets it. Reviewed-by: Brian Paul <brianp@vmware.com> Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
-