[cairo-bugs] [Bug 90303] ATOMIC_OP_NEEDS_MEMORY_BARRIER doesn't play well with ThreadSanitizer

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=90303

--- Comment #7 from Nathan Froyd <froydnj at gmail.com> ---
(In reply to Chris Wilson from comment #5)
> Looks good to me. Something that is missing in that file is that we really
> should do
> 
> #if HAVE_CXX11_ATOMIC_PRIMITIVES
> #elif HAVE_INTEL_ATOMIC_PRIMITIVES
> #elif /* blah */
> #endif

I could do that as a followup patch.  Where do standalone patches like that get
sent?  A bug in bugzilla, or straight to the mailing list?

(In reply to Behdad Esfahbod from comment #6)
> Would you also be able to take a look at improving HarfBuzz (and fontconfig,
> which copied the atomic ops from HarfBuzz; but I can do that part myself):
> 
>   https://bugs.freedesktop.org/show_bug.cgi?id=90305

Sure, I could look at that.  I haven't seen any HarfBuzz bits come up when
running Firefox through TSan, but it's entirely possible that we only use
HarfBuzz on a single thread.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.cairographics.org/archives/cairo-bugs/attachments/20150504/51bda9c1/attachment.html>