Summary: | ValueRecovery should be moved out of the DFG JIT | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Filip Pizlo <fpizlo> | ||||||
Component: | JavaScriptCore | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 71436 | ||||||||
Attachments: |
|
Description
Filip Pizlo
2011-11-02 23:45:26 PDT
Created attachment 113436 [details]
the patch
Attachment 113436 [details] did not pass style-queue:
Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/ChangeLog', u'Source..." exit_code: 1
Source/JavaScriptCore/bytecode/ValueRecovery.h:325: "DataFormat.h" already included at Source/JavaScriptCore/bytecode/ValueRecovery.h:29 [build/include] [4]
Source/JavaScriptCore/bytecode/ValueRecovery.h:326: "JSValue.h" already included at Source/JavaScriptCore/bytecode/ValueRecovery.h:30 [build/include] [4]
Source/JavaScriptCore/bytecode/ValueRecovery.h:327: "MacroAssembler.h" already included at Source/JavaScriptCore/bytecode/ValueRecovery.h:31 [build/include] [4]
Source/JavaScriptCore/bytecode/ValueRecovery.h:328: "VirtualRegister.h" already included at Source/JavaScriptCore/bytecode/ValueRecovery.h:32 [build/include] [4]
Source/JavaScriptCore/bytecode/ValueRecovery.h:329: "wtf/Platform.h" already included at Source/JavaScriptCore/bytecode/ValueRecovery.h:33 [build/include] [4]
Source/JavaScriptCore/bytecode/VirtualRegister.h:69: "wtf/Platform.h" already included at Source/JavaScriptCore/bytecode/VirtualRegister.h:29 [build/include] [4]
Source/JavaScriptCore/ChangeLog:1: ChangeLog entry has no bug number [changelog/bugnumber] [5]
Source/JavaScriptCore/bytecode/DataFormat.h:211: "wtf/Assertions.h" already included at Source/JavaScriptCore/bytecode/DataFormat.h:29 [build/include] [4]
Total errors found: 8 in 8 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 113437 [details]
the patch
Fixed changelog and merge issues.
Landed in http://trac.webkit.org/changeset/99144 |