ChangeLog

2017-03-15  Saam Barati  <sbarati@apple.com>

        WebAssembly: When we GC to try to get a fast memory, we should call collectAllGarbage(), not collectSync()

        https://bugs.webkit.org/show_bug.cgi?id=169704

        Reviewed by NOBODY (OOPS!).

        We weren't always sweeping the memory needed to free

        the WasmMemory we wanted to use. collectAllGarbage()

        will do this if the JS objects wrapping WasmMemory

        are dead.

        This patch also moves the increment of the allocatedFastMemories

        integer to be thread safe.

        * wasm/WasmMemory.cpp:

        (JSC::Wasm::tryGetFastMemory):

2017-03-15  Mark Lam  <mark.lam@apple.com>

        Fix missing exception checks in Interpreter.cpp.

214011

Source/JavaScriptCore/wasm/WasmMemory.cpp

@@inline bool tryGetFastMemory(VM& vm, voi

    // If we have allocated all the fast memories... too bad.

    if (allocatedFastMemories == maxFastMemories) {

        // There is a reasonable chance that another module has died but has not been collected yet. Don't lose hope yet!

122 vm.heap.collect~~Sync~~();

122 vm.heap.collectAllGarbage();

        return dequeFastMemory();

    }

    if (mmapBytes(fastMemoryMappedBytes, memory)) {

        mappedCapacity = fastMemoryMappedBytes;

        mode = Memory::Signaling;

129 allocatedFastMemories++;

130129 LockHolder locker(memoryLock);

130 allocatedFastMemories++;

        auto result = activeFastMemories(locker).add(memory);

        ASSERT_UNUSED(result, result.isNewEntry);

    }

213983