Do inline object filtering (via page flags) before call to RecordWriteStub.

src/ia32/macro-assembler-ia32-inl.h

 // Copyright 2011 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 15 matching lines @@
 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
 #ifndef V8_IA32_MACRO_ASSEMBLER_IA32_INL_H_
 #define V8_IA32_MACRO_ASSEMBLER_IA32_INL_H_
 
 #include "macro-assembler.h"
 
 namespace v8 {
 namespace internal {
 
+
 template<typename LabelType>
-void MacroAssembler::HasScanOnScavenge(Register object,
-                                       Register scratch,
-                                       LabelType* scan_on_scavenge) {
+void MacroAssembler::CheckPageFlag(
+    Register object,
+    Register scratch,
+    MemoryChunk::MemoryChunkFlags flag,
+    Condition cc,
+    LabelType* not_set) {

[Erik Corry, 2011/05/11 18:53:57]

not_set seems like the wrong name, since it means is_set if the condition is
not_zero.  How about condition_met?

[Vyacheslav Egorov (Chromium), 2011/05/13 11:06:52]

Done.

   Move(scratch, object);
   and_(scratch, ~Page::kPageAlignmentMask);
-  cmpb(Operand(scratch, MemoryChunk::kScanOnScavengeOffset), 0);
-  j(not_equal, scan_on_scavenge);
+  test(Operand(scratch, MemoryChunk::kFlagsOffset), Immediate(1 << flag));
+  j(cc, not_set);
 }
 
 
+
 template<typename LabelType>
 void MacroAssembler::InOldSpaceIsBlack(Register object,
                                        Register scratch0,
                                        Register scratch1,
                                        LabelType* is_black) {
   HasColour(object, scratch0, scratch1,
             is_black,
             Page::kPageAlignmentMask,
             MemoryChunk::kHeaderSize,
             1, 0,    // kBlackBitPattern.
@@ skipping 152 matching lines @@
 
   // Value is a data object, and it is white.  Mark it black.  Since we know
   // that the object is white we can make it black by flipping one bit.
   or_(Operand(bitmap_scratch, header_size), mask_scratch);
   bind(&done);
 }
 
 } }  // namespace v8::internal
 
 #endif  // V8_IA32_MACRO_ASSEMBLER_IA32_INL_H_