aaron.ballman added a comment. In D112221#3195506 <https://reviews.llvm.org/D112221#3195506>, @ldionne wrote:
> Sorry, I seemed to have missed this entirely. No worries! > Since this is touching only `<stdatomic.h>` and not libc++'s `<atomic>`, I > think it would actually be OK not to make any changes at all inside libc++. > Nothing should break. Ah, thanks for the info! I've backed the libc++ changes out here. > We should separately implement P0883 in libc++ and deprecate the macros that > we define in `<atomic>`. Note that I thought P0883 was implemented -- it is > definitely marked as such, so we'll have to investigate. CC @tambre who > implemented P0883. > > So, as far as this patch is concerned, I think we could remove all the libc++ > specific stuff and this would LGTM. Thanks for the heads up -- we'll tackle > the C++ equivalent of this change in libc++ separately. Thanks! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D112221/new/ https://reviews.llvm.org/D112221 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
