[webkit-reviews] review granted: [Bug 171404] Fix some RELEASE_ASSERT failures caused by OutOfMemoryErrors. : [Attachment 308484] proposed patch.
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Apr 27 18:07:41 PDT 2017
Saam Barati <sbarati at apple.com> has granted Mark Lam <mark.lam at apple.com>'s
request for review:
Bug 171404: Fix some RELEASE_ASSERT failures caused by OutOfMemoryErrors.
https://bugs.webkit.org/show_bug.cgi?id=171404
Attachment 308484: proposed patch.
https://bugs.webkit.org/attachment.cgi?id=308484&action=review
--- Comment #6 from Saam Barati <sbarati at apple.com> ---
Comment on attachment 308484
--> https://bugs.webkit.org/attachment.cgi?id=308484
proposed patch.
View in context: https://bugs.webkit.org/attachment.cgi?id=308484&action=review
> Source/JavaScriptCore/runtime/JSCellInlines.h:146
> +template<typename T, AllocationFailureMode mode,
GCDeferralContextArgPresense deferralContextArgPresence>
> +void* tryAllocateCellHelper(Heap& heap, GCDeferralContext* deferralContext,
size_t size)
I would mark this as ALWAYS_INLINE
> Source/JavaScriptCore/runtime/JSFixedArray.h:47
> ALWAYS_INLINE static JSFixedArray* createFromArray(ExecState* exec, VM&
vm, JSArray* array)
> {
The caller of this from operationMaterializeObjectInOSR is not OK w/ returning
nullptr here. Please fix to make that release assert.
More information about the webkit-reviews
mailing list