RESOLVED FIXED 203339
Undo incidental change from BytecodeIndex class patch
https://bugs.webkit.org/show_bug.cgi?id=203339
Summary Undo incidental change from BytecodeIndex class patch
Keith Miller
Reported 2019-10-23 14:47:27 PDT
Undo incidental change from BytecodeIndex class patch
Attachments
Patch (2.79 KB, patch)
2019-10-23 14:54 PDT, Keith Miller
no flags
Patch (2.80 KB, patch)
2019-10-23 15:11 PDT, Keith Miller
no flags
Keith Miller
Comment 1 2019-10-23 14:54:12 PDT
Keith Miller
Comment 2 2019-10-23 14:54:59 PDT
Mark Lam
Comment 3 2019-10-23 15:09:00 PDT
Comment on attachment 381733 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=381733&action=review > Source/JavaScriptCore/ChangeLog:11 > + It's not totally clear why we need to claim our bytecode index is > +we can't figure what the true index is. I'd rather unbreak > + our build for now, however, and fix the underlying issue in > + https://bugs.webkit.org/show_bug.cgi?id=203340 Can you fix this?
Keith Miller
Comment 4 2019-10-23 15:10:54 PDT
Comment on attachment 381733 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=381733&action=review >> Source/JavaScriptCore/ChangeLog:11 >> + https://bugs.webkit.org/show_bug.cgi?id=203340 > > Can you fix this? Woah, weird. Sure.
Keith Miller
Comment 5 2019-10-23 15:11:00 PDT
Mark Lam
Comment 6 2019-10-23 15:14:12 PDT
Comment on attachment 381736 [details] Patch r=me if JSC tests are green.
WebKit Commit Bot
Comment 7 2019-10-23 20:09:20 PDT
Comment on attachment 381736 [details] Patch Clearing flags on attachment: 381736 Committed r251525: <https://trac.webkit.org/changeset/251525>
WebKit Commit Bot
Comment 8 2019-10-23 20:09:30 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.