Constant array literals result in unnecessarily large amounts of code
Created attachment 97150 [details] Patch
Comment on attachment 97150 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=97150&action=review > Source/JavaScriptCore/bytecompiler/BytecodeGenerator.cpp:1551 > + unsigned checkLength = 0; This should be #ifndef NDEBUG, or, um, !ASSERT_DISABLED?. Y'know.
Committed r88841: <http://trac.webkit.org/changeset/88841>
(In reply to comment #3) > Committed r88841: <http://trac.webkit.org/changeset/88841> This patch made many tests crash on many platforms. Could you check it?
Any update on fix? We might need to rollout this patch for now if we can't land a fix soon because debug test bots are bailing out due to too many crashes.
Per discussion on IRC, I'm rolling out this patch for now.
FYI, http://build.webkit.org/results/Leopard%20Intel%20Debug%20(Tests)/r88857%20(31299)/results.html http://build.webkit.org/results/SnowLeopard%20Intel%20Debug%20(Tests)/r88853%20(644)/results.html http://build.webkit.org/results/SnowLeopard%20Intel%20Release%20(Tests)/r88848%20(30369)/results.html http://build.webkit.org/results/Windows%207%20Release%20(Tests)/r88848%20(13759)/results.html http://build.webkit.org/results/GTK%20Linux%2032-bit%20Release/r88851%20(14681)/results.html http://build.webkit.org/results/GTK%20Linux%2064-bit%20Debug/r88857%20(23371)/results.html http://build.webkit.org/results/Qt%20Linux%20Release/r88857%20(34128)/results.html
Committed r88873: <http://trac.webkit.org/changeset/88873>