r280607 - PR 27200: Fix names of the atomic lock-free macros.

Richard Smith via cfe-commits cfe-commits at lists.llvm.org
Sat Sep 3 22:06:48 PDT 2016


On 3 Sep 2016 6:01 p.m., "Joerg Sonnenberger via cfe-commits" <
cfe-commits at lists.llvm.org> wrote:

On Sat, Sep 03, 2016 at 05:56:43PM -0700, Richard Smith wrote:
>  /* 7.17.2 Initialization */
>
>  #define ATOMIC_VAR_INIT(value) (value)
>  #define atomic_init __c11_atomic_init
>
> -/* 7.17.3 Order and consistency */
> +/* 7.17.3 Order and consistency */.
>
>
> Stray period here. Do we not have any tests that include this header?

We do. Pre-commit editor snafu. Sorry about that.


Thanks for the fix. Can you add a simple test that we provide macros with
the right name? Other than that, it makes sense to me for this to go to the
release branches. (I'm not sure whether we're doing more 3.8 point releases
now that 3.9 is out though.)


Joerg
_______________________________________________
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/20160903/0cc8c729/attachment.html>


More information about the cfe-commits mailing list