• Clemens Backes's avatar
    [wasm][cleanup] Avoid data race on has_priority_ field · 424e7a06
    Clemens Backes authored
    After https://crrev.com/c/2529140, the actual data race should already
    be fixed. This CL updates documentation (by moving the field to the
    fields protected by {mutex_}), and updates {SetHighPriority} to also
    take the mutex. This change is not strictly necessary, because this
    method is only called right after creating the object, so no other
    threads have access to it yet. But relying on that seems brittle, and
    moving the initialization to the constructor is a bigger refactoring
    that I don't consider worth it at the moment. The whole priority
    management will probably be refactored again soon anyway.
    
    R=ahaas@chromium.org
    
    Bug: v8:11141
    Change-Id: I496b619d551aeb584bd6e777c04ed4df076c3ae9
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2529143Reviewed-by: 's avatarAndreas Haas <ahaas@chromium.org>
    Commit-Queue: Clemens Backes <clemensb@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#71113}
    424e7a06
module-compiler.cc 135 KB