1385438 - AtomicRefCounted doesn't need to use sequential consistency

Status: RESOLVED FIXED

(Core :: MFBT, enhancement, P2)

Description

[Jeff Muizelaar [:jrmuizel]]

See https://doc.rust-lang.org/src/alloc/arc.rs.html#560 and https://stackoverflow.com/questions/28199212/why-does-libcs-implementation-of-shared-ptr-use-full-memory-barriers-instead

Comment 1

[Nathan Froyd [:froydnj]]

Attachment: make mozilla::RefCounted use the strongest memory consistency necessary

We don't need to use sequential consistency for everything, we can get
by with using weaker memory consistency models depending on the operation.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/cde4f36e7c23
make mozilla::RefCounted use the strongest memory consistency necessary; r=jrmuizel

Comment 3

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/cde4f36e7c23