MaterializeNewObject expects the JSFinalObject allocator to be non-null.
Created attachment 331658 [details] Patch
Comment on attachment 331658 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=331658&action=review > Source/JavaScriptCore/ftl/FTLLowerDFGToB3.cpp:9888 > + BlockDirectory* cellAllocator = subspaceFor<JSFinalObject>(vm())->allocatorForNonVirtual(allocationSize, AllocatorForMode::EnsureAllocator); This is safe to do from the compiler thread? I don't think it is. Also, any tests here? How did you find this bug?
I misread the code ignore this...
Yeah, if we didn’t create the allocator by the time we JIT that path, then it’s probably not hot enough to be worth optimizing that much. So, even if the GC did support creating allocators from any thread, I’m not sure doing that here would be better.