[Piglit] [PATCH] Fix test macro-definitions.cl

Tom Stellard tom at stellard.net
Mon Nov 5 16:27:49 PST 2012


On Thu, Nov 01, 2012 at 08:22:06AM -0500, Aaron Watry wrote:
> The original version defined BUILD_OPT, but used BUILD_OPT1.  Clover wasn't capable of passing the test either way.  Now, the test should pass on conforming implementations.
> ---

Reviewed-by: Tom Stellard <thomas.stellard at amd.com>

>  tests/cl/program/build/macro-definitions.cl |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/cl/program/build/macro-definitions.cl b/tests/cl/program/build/macro-definitions.cl
> index a8493b0..1758c3f 100644
> --- a/tests/cl/program/build/macro-definitions.cl
> +++ b/tests/cl/program/build/macro-definitions.cl
> @@ -2,9 +2,9 @@
>  [config]
>  name: Macro Definitions
>  clc_version_min: 10
> -build_options: -D BUILD_OPT -D BUILD_OPT2
> +build_options: -D BUILD_OPT1 -D BUILD_OPT2
>  !*/
>  
>  kernel void dummy_kernel(){
>      int var1 = BUILD_OPT1; int var2 = BUILD_OPT2;
> -}
> \ No newline at end of file
> +}
> -- 
> 1.7.9.5
> 
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/piglit


More information about the Piglit mailing list