RESOLVED FIXED 187915
Add some asserts to help diagnose a crash. 
https://bugs.webkit.org/show_bug.cgi?id=187915
Summary Add some asserts to help diagnose a crash.
Mark Lam
Reported 2018-07-23 11:36:37 PDT
<rdar://problem/42508166>
Attachments
proposed patch. (2.42 KB, patch)
2018-07-23 11:41 PDT, Mark Lam 		msaboff: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Mark Lam
Comment 1 2018-07-23 11:41:28 PDT
Created attachment 345589 [details] proposed patch.
Michael Saboff
Comment 2 2018-07-23 11:43:13 PDT
Comment on attachment 345589 [details] proposed patch. r=me
Saam Barati
Comment 3 2018-07-23 11:59:46 PDT
Comment on attachment 345589 [details] proposed patch. View in context: https://bugs.webkit.org/attachment.cgi?id=345589&action=review > Source/JavaScriptCore/bytecode/CodeBlock.cpp:2424 > + RELEASE_ASSERT(jitType() == JITCode::BaselineJIT); why not WTFCrashWithInfo?
Mark Lam
Comment 4 2018-07-23 12:07:09 PDT
(In reply to Saam Barati from comment #3) > Comment on attachment 345589 [details] > proposed patch. > > View in context: > https://bugs.webkit.org/attachment.cgi?id=345589&action=review > > > Source/JavaScriptCore/bytecode/CodeBlock.cpp:2424 > > + RELEASE_ASSERT(jitType() == JITCode::BaselineJIT); > > why not WTFCrashWithInfo? I just didn't think to use that. I'll change this to: CRASH_WITH_INFO(type);
Mark Lam
Comment 5 2018-07-23 12:29:44 PDT
(In reply to Mark Lam from comment #4) > I just didn't think to use that. I'll change this to: > CRASH_WITH_INFO(type); Make that: CRASH_WITH_INFO(bitwise_cast<uintptr_t>(jitCode().get()), type);
Mark Lam
Comment 6 2018-07-23 12:33:32 PDT
Thanks for the reviews. Landed in r234106: <http://trac.webkit.org/r234106>.
Note You need to log in before you can comment on or make changes to this bug.