[Patch libstdc++] Rewrite cpu/generic/atomic_word.h
Torvald Riegel
triegel@redhat.com
Fri Jun 12 15:24:00 GMT 2015
On Fri, 2015-06-12 at 10:30 +0100, Ramana Radhakrishnan wrote:
> On Fri, Jun 12, 2015 at 10:06 AM, Jonathan Wakely <jwakely@redhat.com> wrote:
> > On 11/06/15 23:56 +0200, Torvald Riegel wrote:
> >>>
> >>> > On Fri, 2015-05-22 at 12:37 +0100, Ramana Radhakrishnan wrote:
> >>> I don't think we can remove _GLIBCXX_READ_MEM_BARRIER and
> >>> _GLIBCXX_WRITE_MEM_BARRIER from atomic_word.h even though they are
> >>> superseded by the atomics as it is published in the documentation as
> >>> available macros.
> >>
> >>
> >> I see. We should at least update the documentation of those, as the
> >> current one isn't a really portable specification. If we can, I'd
> >> deprecate them. Jonathan, what do you think?
> >
> >
> > Yes, I'm in favour of deprecating them. They are GCC-specific anyway,
> > so there is no reason to prefer them to std::atomic_ or __atomic_
> > fences.
>
> I'll treat it as a follow-up.
Thanks.
> Can I get an ack for this patch though ?
The above comment was of a general nature, not specifically about this
patch or meant as a precondition for this patch. Sorry if that wasn't
clear :)
To me, your patch looks good.
More information about the Gcc-patches
mailing list