Generalize HeapObject alignment requirements.

src/objects-debug.cc

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "src/v8.h"
 
 #include "src/disasm.h"
 #include "src/disassembler.h"
 #include "src/heap/objects-visiting.h"
 #include "src/jsregexp.h"
@@ skipping 29 matching lines @@
 
 
 void HeapObject::HeapObjectVerify() {
   InstanceType instance_type = map()->instance_type();
 
   if (instance_type < FIRST_NONSTRING_TYPE) {
     String::cast(this)->StringVerify();
     return;
   }
 
   // TODO(yangguo): Use this check once crbug/436911 has been fixed.

[Hannes Payer (out of office), 2015/05/21 09:53:31]

That should be fixed now, we can remove the uncommented code.

[bbudge, 2015/05/21 10:02:07]

Great, Done.

-  //  DCHECK(!NeedsToEnsureDoubleAlignment() ||
+  //  DCHECK(RequiredAlignment() != kDoubleAlignment ||
   //         IsAligned(OffsetFrom(address()), kDoubleAlignment));
 
   switch (instance_type) {
     case SYMBOL_TYPE:
       Symbol::cast(this)->SymbolVerify();
       break;
     case MAP_TYPE:
       Map::cast(this)->MapVerify();
       break;
     case HEAP_NUMBER_TYPE:
@@ skipping 1218 matching lines @@
                          ? it.rinfo()->target_cell()
                          : it.rinfo()->target_object();
     CHECK(!CanLeak(target, heap, skip_weak_cell));
   }
 }
 
 
 #endif  // DEBUG
 
 } }  // namespace v8::internal