r231280 - Fix test/CodeGen/builtins.c for platforms that don't lower sjlj

Joerg Sonnenberger joerg at britannica.bec.de
Wed Mar 4 13:57:22 PST 2015


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



More information about the cfe-commits mailing list