[Mesa-dev] libclc: Improve libclc handling of built-in functions
Tom Stellard
tom at stellard.net
Fri Mar 15 12:06:14 PDT 2013
On Thu, Mar 14, 2013 at 10:01:00PM -0500, Aaron Watry wrote:
> This series depends on the one-off patch I just sent to add max().
>
> 1) Fix the broken abs_diff integer built-in.
> 2) Add clamp for both integer and floating types in a new shared/ dir in order
> to reduce code duplication and improve maintainability.
> 3) Move the max() function into the shared/ directory.
This series looks good and actually negates my comment on the max patch.
I think we should make sure we have piglit tests that cover the NAN
and INF cases, because I'm not sure we handle those correctly in the
backend.
For the series:
Reviewed-by: Tom Stellard <thomas.stellard at amd.com>
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
More information about the mesa-dev
mailing list