Summary: | [JSC] Enable LLInt ASM interpreter on X64 and ARM64 in non JIT configuration | ||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Yusuke Suzuki <ysuzuki> | ||||||||||||||||
Component: | JavaScriptCore | Assignee: | Yusuke Suzuki <ysuzuki> | ||||||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||||||
Severity: | Normal | CC: | ews-watchlist, fpizlo, keith_miller, mark.lam, msaboff, saam, webkit-bug-importer | ||||||||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||||||||
Version: | WebKit Nightly Build | ||||||||||||||||||
Hardware: | Unspecified | ||||||||||||||||||
OS: | Unspecified | ||||||||||||||||||
Attachments: |
|
Description
Yusuke Suzuki
2018-09-19 22:46:13 PDT
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> |