...
Created attachment 214704 [details] work in progress
Created attachment 214706 [details] the patch Heh. It seems to work.
Attachment 214706 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/CMakeLists.txt', u'Source/JavaScriptCore/ChangeLog', u'Source/JavaScriptCore/GNUmakefile.list.am', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.vcxproj', u'Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj', u'Source/JavaScriptCore/assembler/MacroAssembler.h', u'Source/JavaScriptCore/bytecode/StructureStubInfo.h', u'Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp', u'Source/JavaScriptCore/dfg/DFGSpeculativeJIT.h', u'Source/JavaScriptCore/ftl/FTLSaveRestore.cpp', u'Source/JavaScriptCore/jit/JITInlineCacheGenerator.cpp', u'Source/JavaScriptCore/jit/JITInlineCacheGenerator.h', u'Source/JavaScriptCore/jit/JITPropertyAccess.cpp', u'Source/JavaScriptCore/jit/JITPropertyAccess32_64.cpp', u'Source/JavaScriptCore/jit/RegisterSet.cpp', u'Source/JavaScriptCore/jit/RegisterSet.h', u'Source/JavaScriptCore/jit/Repatch.cpp', u'Source/JavaScriptCore/jit/TempRegisterSet.cpp', u'Source/JavaScriptCore/jit/TempRegisterSet.h', u'Source/WTF/ChangeLog', u'Source/WTF/wtf/BitVector.cpp', u'Source/WTF/wtf/BitVector.h']" exit_code: 1 Source/JavaScriptCore/bytecode/StructureStubInfo.h:41: Alphabetical sorting problem. [build/include_order] [4] Source/JavaScriptCore/jit/JITInlineCacheGenerator.h:59: The parameter name "value" adds no information, so it should be removed. [readability/parameter_name] [5] Total errors found: 2 in 22 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 214706 [details] the patch View in context: https://bugs.webkit.org/attachment.cgi?id=214706&action=review >> Source/JavaScriptCore/bytecode/StructureStubInfo.h:41 >> +#include "RegisterSet.h" > > Alphabetical sorting problem. [build/include_order] [4] Sort please.
(In reply to comment #4) > (From update of attachment 214706 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=214706&action=review > > >> Source/JavaScriptCore/bytecode/StructureStubInfo.h:41 > >> +#include "RegisterSet.h" > > > > Alphabetical sorting problem. [build/include_order] [4] > > Sort please. Fixed!
Landed in http://trac.webkit.org/changeset/157707
(In reply to comment #6) > Landed in http://trac.webkit.org/changeset/157707 FYI: It broke the Apple Windows build: 1>JITPropertyAccess32_64.obj : error LNK2019: unresolved external symbol "public: static class JSC::RegisterSet __cdecl JSC::RegisterSet::specialRegisters(void)" (?specialRegisters@RegisterSet@JSC@@SA?AV12@XZ) referenced in function "private: void __thiscall JSC::JIT::emit_op_get_by_id(struct JSC::Instruction *)" (?emit_op_get_by_id@JIT@JSC@@AAEXPAUInstruction@2@@Z) 1>Repatch.obj : error LNK2019: unresolved external symbol "public: __thiscall JSC::TempRegisterSet::TempRegisterSet(class JSC::RegisterSet const &)" (??0TempRegisterSet@JSC@@QAE@ABVRegisterSet@1@@Z) referenced in function "void __cdecl JSC::generateProtoChainAccessStub(class JSC::ExecState *,struct JSC::StructureStubInfo &,class JSC::StructureChain *,unsigned int,int,class JSC::Structure *,class JSC::CodeLocationLabel,class JSC::CodeLocationLabel,class WTF::RefPtr<class JSC::JITStubRoutine> &)" (?generateProtoChainAccessStub@JSC@@YAXPAVExecState@1@AAUStructureStubInfo@1@PAVStructureChain@1@IHPAVStructure@1@VCodeLocationLabel@1@4AAV?$RefPtr@VJITStubRoutine@JSC@@@WTF@@@Z) 1>C:\cygwin\home\buildbot\slave\win-release\build\WebKitBuild\Release\bin32\JavaScriptCore.dll : fatal error LNK1120: 2 unresolved externals 1>Done Building Project "C:\cygwin\home\buildbot\slave\win-release\build\Source\JavaScriptCore\JavaScriptCore.vcxproj\JavaScriptCore.vcxproj" (build target(s)) -- FAILED.