Bug 1482178 - IonBuilder needs to access script zone from any thread, r=jandem
authorSteve Fink <sfink@mozilla.com>
Wed, 22 Aug 2018 10:48:27 -0700
changeset 490805 c420e3bec25bbdc2980353017f775acae3e12ff3
parent 490804 ae7dc05bee296f01c98e9b6d166ab308ea286690
child 490806 4c20ba876af1c686aa454d8e8e216828df04216a
push id1815
push userffxbld-merge
push dateMon, 15 Oct 2018 10:40:45 +0000
treeherdermozilla-release@18d4c09e9378 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjandem
bugs1482178
milestone63.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1482178 - IonBuilder needs to access script zone from any thread, r=jandem
js/src/vm/HelperThreads.cpp
--- a/js/src/vm/HelperThreads.cpp
+++ b/js/src/vm/HelperThreads.cpp
@@ -274,17 +274,17 @@ static bool
 IonBuilderMatches(const CompilationSelector& selector, jit::IonBuilder* builder)
 {
     struct BuilderMatches
     {
         jit::IonBuilder* builder_;
 
         bool match(JSScript* script)    { return script == builder_->script(); }
         bool match(Realm* realm)        { return realm == builder_->script()->realm(); }
-        bool match(Zone* zone)          { return zone == builder_->script()->zone(); }
+        bool match(Zone* zone)          { return zone == builder_->script()->zoneFromAnyThread(); }
         bool match(JSRuntime* runtime)  { return runtime == builder_->script()->runtimeFromAnyThread(); }
         bool match(AllCompilations all) { return true; }
         bool match(ZonesInState zbs)    {
             return zbs.runtime == builder_->script()->runtimeFromAnyThread() &&
                    zbs.state == builder_->script()->zoneFromAnyThread()->gcState();
         }
         bool match(CompilationsUsingNursery cun) {
             return cun.runtime == builder_->script()->runtimeFromAnyThread() &&