On Wed, Mar 04, 2015 at 07:24:17PM -0000, Reid Kleckner wrote:
> Modified: cfe/trunk/test/CodeGen/builtins.c
> URL: 
> http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGen/builtins.c?rev=231280&r1=231279&r2=231280&view=diff
> ==============================================================================
> --- cfe/trunk/test/CodeGen/builtins.c (original)
> +++ cfe/trunk/test/CodeGen/builtins.c Wed Mar  4 13:24:16 2015
> @@ -220,6 +220,8 @@ void test_float_builtin_ops(float F, dou
>    // CHECK: call x86_fp80 @llvm.fabs.f80(x86_fp80
>  }
>  
> +// __builtin_longjmp isn't supported on all platforms, so only test it on 
> X86.
> +#ifdef __x86_64__
>  // CHECK-LABEL: define void @test_builtin_longjmp
>  void test_builtin_longjmp(void **buffer) {
>    // CHECK: [[BITCAST:%.*]] = bitcast
> @@ -227,6 +229,7 @@ void test_builtin_longjmp(void **buffer)
>    __builtin_longjmp(buffer, 1);
>    // CHECK-NEXT: unreachable
>  }
> +#endif
>  
>  // CHECK-LABEL: define i64 @test_builtin_readcyclecounter
>  long long test_builtin_readcyclecounter() {

I think we should just drop the test here, it is more extensively tested
in a separate file?

Joerg
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to