walkero
/
harfbuzz
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix previous commit to use atomic operations

This commit is contained in:
Behdad Esfahbod 2018-08-08 22:01:04 -07:00
parent 883957688c
commit 51ffc3e65a
1 changed files with 10 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/hb-machinery-private.hh
View File

@ -639,9 +639,16 @@ struct hb_lazy_loader_t
inline void set_stored (Stored *instance_) inline void set_stored (Stored *instance_)
{ {
if (instance) /* This *must* be called when there are no other threads accessing.
thiz ()->destroy (instance); * However, to make TSan, etc, happy, we using cmpexch. */
instance = instance_; retry:
Stored *p = (Stored *) hb_atomic_ptr_get (&this->instance);
if (p)
{
if (unlikely (!hb_atomic_ptr_cmpexch (const_cast<Stored **>(&this->instance), p, instance_)))
goto retry;
thiz ()->destroy (p);
}
} }
inline const Returned * get (void) const inline const Returned * get (void) const