[JSC] Add more JSType based fast path for jsDynamicCast
Created attachment 335183 [details] Patch
Comment on attachment 335183 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=335183&action=review Nice work. r=me if you resolve EWS failures. > Source/JavaScriptCore/runtime/JSObject.h:1100 > // storage to fully make use of the colloctor cell containing it. Can you fix this typo as well: /colloctor/collector/. > Source/JavaScriptCore/runtime/JSScope.h:43 > + DECLARE_INFO; I suspect that you need to use DECLARE_EXPORT_INFO here (see EWS build failure).
Comment on attachment 335183 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=335183&action=review Thanks! >> Source/JavaScriptCore/runtime/JSObject.h:1100 >> // storage to fully make use of the colloctor cell containing it. > > Can you fix this typo as well: /colloctor/collector/. Fixed. Also I fixed /sufficent/sufficient/. >> Source/JavaScriptCore/runtime/JSScope.h:43 >> + DECLARE_INFO; > > I suspect that you need to use DECLARE_EXPORT_INFO here (see EWS build failure). Yeah, I think so. I'll upload "Patch for landing" to check it on EWS :).
Created attachment 335192 [details] Patch
Comment on attachment 335192 [details] Patch Oops, cleared r+.
OK, let's land it :D
Committed r229362: <https://trac.webkit.org/changeset/229362>
<rdar://problem/38224191>
Comment on attachment 335192 [details] Patch Nice!