Ignore:
Timestamp:
Jan 22, 2011, 12:11:22 PM (14 years ago)
Author:
[email protected]
Message:

2011-01-22 Geoffrey Garen <[email protected]>

Reviewed by Geoffrey Garen.

Rubber-stamped by Maciej Stachowiak.

A few of Maciej's review suggestions for my last patch.
https://bugs.webkit.org/show_bug.cgi?id=52946

SunSpider reports no change.

  • runtime/ConservativeSet.cpp: Added. (JSC::isPointerAligned): (JSC::ConservativeSet::add):
  • runtime/ConservativeSet.h: Added. (JSC::ConservativeSet::ConservativeSet): (JSC::ConservativeSet::mark): Split ConservativeSet out into its own file, and moved the conservative check into ConservativeSet::add, making ConservativeSet's responsibility clearer.
  • runtime/Heap.cpp: (JSC::Heap::markRoots):
  • runtime/MachineStackMarker.cpp: (JSC::MachineStackMarker::markCurrentThreadConservativelyInternal): (JSC::MachineStackMarker::markOtherThreadConservatively):
  • runtime/MachineStackMarker.h:
  • runtime/MarkStack.h: Updated for changes above.
File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/JavaScriptCore/JavaScriptCore.gypi

    r76185 r76454  
    281281            'runtime/MachineStackMarker.cpp',
    282282            'runtime/MachineStackMarker.h',
     283            'runtime/ConservativeSet.cpp',
     284            'runtime/ConservativeSet.h',
    283285            'runtime/MarkStack.cpp',
    284286            'runtime/MarkStack.h',
Note: See TracChangeset for help on using the changeset viewer.