Makes sense. Reviewed-by: Jason Ekstrand <ja...@jlekstrand.net>
On Tue, May 29, 2018 at 12:07 AM, Samuel Iglesias Gonsálvez < sigles...@igalia.com> wrote: > If the application asks for the maximum number of fragment input > components (128), use all of them plus some builtins that are > passed in the VUE, then we exceed the maximum number of used VUE > slots (32) and we break one assert that checks this limit. > > Also, with separate shader objects, we add CLIP_DIST0, CLIP_DIST1 > builtins in brw_compute_vue_map() because we don't know if > gl_ClipDistance is going to be read/write by an adjacent stage. > > Fixes VK-GL-CTS CL#2569. > > Signed-off-by: Samuel Iglesias Gonsálvez <sigles...@igalia.com> > --- > src/intel/vulkan/anv_device.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/intel/vulkan/anv_device.c b/src/intel/vulkan/anv_device.c > index 374fc16c4c9..87c0d0cb4a6 100644 > --- a/src/intel/vulkan/anv_device.c > +++ b/src/intel/vulkan/anv_device.c > @@ -898,7 +898,7 @@ void anv_GetPhysicalDeviceProperties( > .maxGeometryOutputComponents = 128, > .maxGeometryOutputVertices = 256, > .maxGeometryTotalOutputComponents = 1024, > - .maxFragmentInputComponents = 128, > + .maxFragmentInputComponents = 112, /* 128 components > - (POS, PSIZ, CLIP_DIST0, CLIP_DIST1) */ > .maxFragmentOutputAttachments = 8, > .maxFragmentDualSrcAttachments = 1, > .maxFragmentCombinedOutputResources = 8, > -- > 2.17.0 > >
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
