LLInt ASM interpreter is 2x faster than CLoop interpreter. We would like to enable it for non JIT configuration in x64 and ARM64. I don't consider enabling this for the other CPU architectures now since no buildbot exists for non JIT configurations.
Background: https://lists.webkit.org/pipermail/webkit-dev/2018-September/030157.html
Created attachment 350170 [details] Patch
Created attachment 350338 [details] Patch
Attachment 350338 [details] did not pass style-queue: ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:57: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:73: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:86: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:97: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:108: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:130: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] Total errors found: 6 in 56 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 350340 [details] Patch
Attachment 350340 [details] did not pass style-queue: ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:57: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:73: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:86: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:97: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:108: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:130: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] Total errors found: 6 in 57 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 350342 [details] Patch
Attachment 350342 [details] did not pass style-queue: ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:57: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:73: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:86: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:97: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:108: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:130: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] Total errors found: 6 in 58 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 350355 [details] Patch
Attachment 350355 [details] did not pass style-queue: ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:57: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:73: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:86: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:97: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:108: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:130: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] Total errors found: 6 in 58 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 350355 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=350355&action=review r=me with some nits. > Source/JavaScriptCore/ChangeLog:10 > + configuration in X86_64 and ARM64. Nit: configurations > Source/JavaScriptCore/ChangeLog:12 > + This patch enables LLInt for non JIT in X86_64 and ARM64 architectures. Nit: maybe "for non JIT in" => "for non JIT builds on" > Source/JavaScriptCore/ChangeLog:14 > + configuration. But it is wrong in the new scenario since we have a buildx typo: "buildx" => "build"? > Source/JavaScriptCore/jit/HostCallReturnValue.cpp:-38 > -// Nothing to see here. Lol solid comment.
Comment on attachment 350355 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=350355&action=review Thank you! >> Source/JavaScriptCore/ChangeLog:10 >> + configuration in X86_64 and ARM64. > > Nit: configurations Fixed. >> Source/JavaScriptCore/ChangeLog:12 >> + This patch enables LLInt for non JIT in X86_64 and ARM64 architectures. > > Nit: maybe "for non JIT in" => "for non JIT builds on" Fixed. >> Source/JavaScriptCore/ChangeLog:14 >> + configuration. But it is wrong in the new scenario since we have a buildx > > typo: "buildx" => "build"? Fixed.
Created attachment 350480 [details] Patch
Created attachment 350482 [details] Patch
Attachment 350480 [details] did not pass style-queue: ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:57: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:73: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:86: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:97: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:108: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:130: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] Total errors found: 6 in 57 files If any of these errors are false positives, please file a bug against check-webkit-style.
Attachment 350482 [details] did not pass style-queue: ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:57: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:73: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:86: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:97: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:108: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/jit/HostCallReturnValue.cpp:130: The parameter name """" adds no information, so it should be removed. [readability/parameter_name] [5] Total errors found: 6 in 57 files If any of these errors are false positives, please file a bug against check-webkit-style.
Committed r236381: <https://trac.webkit.org/changeset/236381>
<rdar://problem/44701303>