[prev in list] [next in list] [prev in thread] [next in thread]
List: mesa3d-dev
Subject: Re: [Mesa-dev] [PATCH] linker: Fix consumer_inputs_with_locations indexing
From: Kenneth Graunke <kenneth () whitecape ! org>
Date: 2014-05-05 21:44:17
Message-ID: 53680631.9050903 () whitecape ! org
[Download RAW message or body]
[Attachment #2 (multipart/signed)]
On 05/05/2014 10:56 AM, Ian Romanick wrote:
> From: Ian Romanick <ian.d.romanick@intel.com>
>
> In an earlier incarnation of populate_consumer_input_sets and
> get_matching_input, the consumer_inputs_with_locations array was indexed
> using the user-specified location. In that version, only user-defined
> varyings were included in the array.
>
> In the current incarnation, the Mesa location is used to index the
> array, and built-in varyings are included.
>
> This change fixes the unit test to exepect gl_ClipDistance in the array,
> and it resizes the arrays to actually be big enough. It's just dumb
> luck that the existing piglit tests use small enough locations to not
> stomp the stack. :(
>
> Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=78258
> Cc: "10.2" <mesa-stable@lists.freedesktop.org>
> Cc: Vinson Lee <vlee@freedesktop.org>
> ---
> src/glsl/link_varyings.cpp | 8 ++++----
> src/glsl/tests/varyings_test.cpp | 11 +++++------
> 2 files changed, 9 insertions(+), 10 deletions(-)
Looks reasonable to me. Presumably this is new code, so we don't need
to backport beyond 10.2.
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
["signature.asc" (application/pgp-signature)]
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic