r336658 - Fixing builtin __atomic_fetch_min declaration

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 10 05:11:53 PDT 2018


On Tue, Jul 10, 2018 at 8:04 AM, Omer Paparo Bivas via cfe-commits
<cfe-commits at lists.llvm.org> wrote:
> Author: opaparo
> Date: Tue Jul 10 05:04:04 2018
> New Revision: 336658
>
> URL: http://llvm.org/viewvc/llvm-project?rev=336658&view=rev
> Log:
> Fixing builtin __atomic_fetch_min declaration

Can a test case also be committed for this change?

~Aaron

>
> Differential Revision: http://reviews.llvm.org/D49068
>
> Modified:
>     cfe/trunk/include/clang/Basic/Builtins.def
>
> Modified: cfe/trunk/include/clang/Basic/Builtins.def
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/Builtins.def?rev=336658&r1=336657&r2=336658&view=diff
> ==============================================================================
> --- cfe/trunk/include/clang/Basic/Builtins.def (original)
> +++ cfe/trunk/include/clang/Basic/Builtins.def Tue Jul 10 05:04:04 2018
> @@ -723,7 +723,7 @@ ATOMIC_BUILTIN(__opencl_atomic_fetch_min
>  ATOMIC_BUILTIN(__opencl_atomic_fetch_max, "v.", "t")
>
>  // GCC does not support these, they are a Clang extension.
> -ATOMIC_BUILTIN(__atomic_fetch_min, "iiD*i.", "t")
> +ATOMIC_BUILTIN(__atomic_fetch_min, "v.", "t")
>  ATOMIC_BUILTIN(__atomic_fetch_max, "v.", "t")
>
>  #undef ATOMIC_BUILTIN
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits


More information about the cfe-commits mailing list