Summary: | Renamed JSGlobalData to VM | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Geoffrey Garen <ggaren> | ||||||||||
Component: | New Bugs | Assignee: | Geoffrey Garen <ggaren> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | eflews.bot, gyuyoung.kim | ||||||||||
Priority: | P2 | ||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||
Hardware: | Unspecified | ||||||||||||
OS: | Unspecified | ||||||||||||
Attachments: |
|
Description
Geoffrey Garen
2013-04-17 17:48:38 PDT
Created attachment 198635 [details]
Patch
Comment on attachment 198635 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=198635&action=review > Source/JavaScriptCore/JavaScriptCore.order:26 > +__ZN3JSC12VM10storeVPtrsEv Shouldn't this be __ZN3JSC2VM10storeVPtrsEv (the 12 preceding JSGlobalData is now 2 preceding VM)? > Source/JavaScriptCore/JavaScriptCore.order:123 > -__ZN3JSC12JSGlobalData14sharedInstanceEv > -__ZN3JSC12JSGlobalDataC2ENS0_14GlobalDataTypeENS_8HeapTypeE > +__ZN3JSC12VM14sharedInstanceEv > +__ZN3JSC12VMC2ENS0_14GlobalDataTypeENS_8HeapTypeE Ditto. 12 ==> 2. > Source/JavaScriptCore/JavaScriptCore.order:127 > -__ZN3JSC17CommonIdentifiersC1EPNS_12JSGlobalDataE > -__ZN3JSC17CommonIdentifiersC2EPNS_12JSGlobalDataE > -__ZN3JSC10Identifier3addEPNS_12JSGlobalDataEPKc > +__ZN3JSC17CommonIdentifiersC1EPNS_12VME > +__ZN3JSC17CommonIdentifiersC2EPNS_12VME > +__ZN3JSC10Identifier3addEPNS_12VMEPKc 12 => 2. > Source/JavaScriptCore/JavaScriptCore.order:132 > -__ZN3JSC10IdentifierC1EPNS_12JSGlobalDataEPKc > +__ZN3JSC10IdentifierC1EPNS_12VMEPKc 12 => 2. > Source/JavaScriptCore/JavaScriptCore.order:139 > -__ZN3JSC5LexerC1EPNS_12JSGlobalDataE > +__ZN3JSC5LexerC1EPNS_12VME 12 => 2. > Source/JavaScriptCore/JavaScriptCore.order:145 > -__ZN3JSC4HeapC1EPNS_12JSGlobalDataE > -__ZN3JSC4HeapC2EPNS_12JSGlobalDataE > -__ZN3JSC11MarkedSpaceC1EPNS_12JSGlobalDataE > +__ZN3JSC4HeapC1EPNS_12VME > +__ZN3JSC4HeapC2EPNS_12VME > +__ZN3JSC11MarkedSpaceC1EPNS_12VME 12 => 2. > Source/JavaScriptCore/JavaScriptCore.order:151 > -__ZN3JSC10HandleHeapC1EPNS_12JSGlobalDataE > +__ZN3JSC10HandleHeapC1EPNS_12VME 12 => 2. > Source/JavaScriptCore/JavaScriptCore.order:159 > -__ZN3JSC11RegExpCacheC1EPNS_12JSGlobalDataE > -__ZN3JSC11InterpreterC1ERNS_12JSGlobalDataE > +__ZN3JSC11RegExpCacheC1EPNS_12VME > +__ZN3JSC11InterpreterC1ERNS_12VME 12 => 2. > Source/JavaScriptCore/JavaScriptCore.order:164 > -__ZN3JSC11MarkedBlock6createEPNS_12JSGlobalDataEm > +__ZN3JSC11MarkedBlock6createEPNS_12VMEm 12 => 2. > Source/JavaScriptCore/JavaScriptCore.order:171 > -__ZN3JSC9StructureC1ERNS_12JSGlobalDataE > -__ZN3JSC6StrongINS_9StructureEE3setERNS_12JSGlobalDataEPS1_ > -__ZN3JSC9Structure6createERNS_12JSGlobalDataENS_7JSValueERKNS_8TypeInfoEjPKNS_9ClassInfoE > -__ZN3JSC9StructureC1ERNS_12JSGlobalDataENS_7JSValueERKNS_8TypeInfoEjPKNS_9ClassInfoE > +__ZN3JSC9StructureC1ERNS_12VME > +__ZN3JSC6StrongINS_9StructureEE3setERNS_12VMEPS1_ > +__ZN3JSC9Structure6createERNS_12VMENS_7JSValueERKNS_8TypeInfoEjPKNS_9ClassInfoE > +__ZN3JSC9StructureC1ERNS_12VMENS_7JSValueERKNS_8TypeInfoEjPKNS_9ClassInfoE 12 => 2. > Source/JavaScriptCore/JavaScriptCore.order:180 > -__ZN3JSC9JITThunksC1EPNS_12JSGlobalDataE > -__ZN3JSC9JITThunksC2EPNS_12JSGlobalDataE > -__ZN3JSC3JITC1EPNS_12JSGlobalDataEPNS_9CodeBlockE > -__ZN3JSC3JITC2EPNS_12JSGlobalDataEPNS_9CodeBlockE > -__ZN3JSC3JIT35privateCompileCTIMachineTrampolinesEPN3WTF6RefPtrINS_14ExecutablePoolEEEPNS_12JSGlobalDataEPNS_19TrampolineStructureE > +__ZN3JSC9JITThunksC1EPNS_12VME > +__ZN3JSC9JITThunksC2EPNS_12VME > +__ZN3JSC3JITC1EPNS_12VMEPNS_9CodeBlockE > +__ZN3JSC3JITC2EPNS_12VMEPNS_9CodeBlockE > +__ZN3JSC3JIT35privateCompileCTIMachineTrampolinesEPN3WTF6RefPtrINS_14ExecutablePoolEEEPNS_12VMEPNS_19TrampolineStructureE 12 => 2. > Source/JavaScriptCore/JavaScriptCore.order:195 > -__ZN3JSC3JIT27privateCompileCTINativeCallEPNS_12JSGlobalDataEb > +__ZN3JSC3JIT27privateCompileCTINativeCallEPNS_12VMEb 12 => 2. > Source/JavaScriptCore/JavaScriptCore.order:203 > -__ZN3JSC14JSGlobalObjectnwEmPNS_12JSGlobalDataE > +__ZN3JSC14JSGlobalObjectnwEmPNS_12VME 12 => 2. > Source/JavaScriptCore/JavaScriptCore.order:211 > -__ZN3JSC16InternalFunctionC2EPNS_12JSGlobalDataEPNS_14JSGlobalObjectEPNS_9StructureERKNS_10IdentifierE > +__ZN3JSC16InternalFunctionC2EPNS_12VMEPNS_14JSGlobalObjectEPNS_9StructureERKNS_10IdentifierE 12 => 2. And so on below ... > > Source/JavaScriptCore/JavaScriptCore.order:26
> > +__ZN3JSC12VM10storeVPtrsEv
>
> Shouldn't this be __ZN3JSC2VM10storeVPtrsEv (the 12 preceding JSGlobalData is now 2 preceding VM)?
You're probably right, but it's OK for the .order file to be wrong for now. It's just an optimization, not a correctness issue, and we re-auto-generate it for each release.
Created attachment 198674 [details]
Patch
Comment on attachment 198674 [details] Patch Attachment 198674 [details] did not pass efl-ews (efl): Output: http://webkit-queues.appspot.com/results/172113 Created attachment 198675 [details]
Patch
Created attachment 198746 [details]
Patch
Committed r148696: <http://trac.webkit.org/changeset/148696> Comment on attachment 198746 [details]
Patch
Looks like this is landed already.
|