Thanks, I uploaded new patch set.
https://chromiumcodereview.appspot.com/12094036/diff/2001/src/mark-compact.cc
File src/mark-compact.cc (right):
https://chromiumcodereview.appspot.com/12094036/diff/2001/src/mark-compact.cc#newcode837
src/mark-compact.cc:837: static void
VerifyWeakEmbeddedMapsInOptimizedCode(Heap* heap) {
On 2013/01/31 14:27:50, Michael Starzinger wrote:
Move this function up to the other verifiers at the top of the file.
Done.
https://chromiumcodereview.appspot.com/12094036/diff/2001/src/mark-compact.cc#newcode870
src/mark-compact.cc:870: if (FLAG_collect_maps &&
FLAG_weak_embedded_maps_in_optimized_code &&
On 2013/01/31 14:27:50, Michael Starzinger wrote:
Move this call into MarkCompactCollector::CollectGarbage() where the
other
verifiers are called.
Done.
https://chromiumcodereview.appspot.com/12094036/diff/2001/src/mark-compact.cc#newcode2344
src/mark-compact.cc:2344: number_of_codes = codes->number_of_codes();
On 2013/01/31 14:27:50, Michael Starzinger wrote:
This call seems to be obsolete.
Done.
https://chromiumcodereview.appspot.com/12094036/diff/2001/src/objects-inl.h
File src/objects-inl.h (right):
https://chromiumcodereview.appspot.com/12094036/diff/2001/src/objects-inl.h#newcode80
src/objects-inl.h:80: ASSERT(object->Is##type()); \
On 2013/01/31 14:27:50, Michael Starzinger wrote:
Unnecessary white-space change.
Done.
https://chromiumcodereview.appspot.com/12094036/diff/2001/src/objects.cc
File src/objects.cc (right):
https://chromiumcodereview.appspot.com/12094036/diff/2001/src/objects.cc#newcode9212
src/objects.cc:9212: void Code::VerifyEmbeddedMaps() {
On 2013/01/31 14:27:50, Michael Starzinger wrote:
This whole implementation belongs into objects-debug.cc I guess.
Done.
https://chromiumcodereview.appspot.com/12094036/diff/2001/src/objects.cc#newcode9218
src/objects.cc:9218: Map* map = Map::cast(it.rinfo()->target_object());
On 2013/01/31 14:27:50, Michael Starzinger wrote:
Indentation is off.
Done.
https://chromiumcodereview.appspot.com/12094036/diff/2001/src/objects.h
File src/objects.h (right):
https://chromiumcodereview.appspot.com/12094036/diff/2001/src/objects.h#newcode4549
src/objects.h:4549: void VerifyEmbeddedMaps();
On 2013/01/31 14:27:50, Michael Starzinger wrote:
Let's call this "VerifyEmbeddedMapsDependency()".
Done.
https://chromiumcodereview.appspot.com/12094036/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
