r285027 - Fix test on non-X86 platforms

[Craig Topper via cfe-commits]

Doesn't this exclude 32-bit x86?

~Craig

On Mon, Oct 24, 2016 at 2:22 PM, Mehdi Amini via cfe-commits <
cfe-commits at lists.llvm.org> wrote:

> Author: mehdi_amini
> Date: Mon Oct 24 16:22:01 2016
> New Revision: 285027
>
> URL: http://llvm.org/viewvc/llvm-project?rev=285027&view=rev
> Log:
> Fix test on non-X86 platforms
>
> This is a fixup for r285019, adding an `#ifdef __x86_64__` since
> the os_log builtin is platform specific.
>
> Modified:
>     cfe/trunk/test/CodeGen/builtins.c
>
> Modified: cfe/trunk/test/CodeGen/builtins.c
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/
> CodeGen/builtins.c?rev=285027&r1=285026&r2=285027&view=diff
> ============================================================
> ==================
> --- cfe/trunk/test/CodeGen/builtins.c (original)
> +++ cfe/trunk/test/CodeGen/builtins.c Mon Oct 24 16:22:01 2016
> @@ -369,6 +369,9 @@ long long test_builtin_readcyclecounter(
>    return __builtin_readcyclecounter();
>  }
>
> +// Behavior of __builtin_os_log differs between platforms, so only test
> on X86
> +#ifdef __x86_64__
> +
>  // CHECK-LABEL: define void @test_builtin_os_log
>  // CHECK: (i8* [[BUF:%.*]], i32 [[I:%.*]], i8* [[DATA:%.*]])
>  void test_builtin_os_log(void *buf, int i, const char *data) {
> @@ -506,3 +509,5 @@ void test_builtin_os_log_percent(void *b
>    // CHECK: store i8* [[DATA2]], i8** [[ARG1_PTR]]
>    __builtin_os_log_format(buf, "%s %%", data);
>  }
> +
> +#endif
> \ No newline at end of file
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20161024/4020fea3/attachment.html>