Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(9)

Side by Side Diff: src/heap/heap.cc

Issue 1131123003: Fix Heap::AllocateFillerObject() to use double alignment param. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "src/v8.h" 5 #include "src/v8.h"
6 6
7 #include "src/accessors.h" 7 #include "src/accessors.h"
8 #include "src/api.h" 8 #include "src/api.h"
9 #include "src/base/bits.h" 9 #include "src/base/bits.h"
10 #include "src/base/once.h" 10 #include "src/base/once.h"
(...skipping 2516 matching lines...) Expand 10 before | Expand all | Expand 10 after
2527 map->set_elements_kind(elements_kind); 2527 map->set_elements_kind(elements_kind);
2528 2528
2529 return map; 2529 return map;
2530 } 2530 }
2531 2531
2532 2532
2533 AllocationResult Heap::AllocateFillerObject(int size, bool double_align, 2533 AllocationResult Heap::AllocateFillerObject(int size, bool double_align,
2534 AllocationSpace space) { 2534 AllocationSpace space) {
2535 HeapObject* obj; 2535 HeapObject* obj;
2536 { 2536 {
2537 AllocationResult allocation = AllocateRaw(size, space, space); 2537 AllocationAlignment align = double_align ? kDoubleAligned : kWordAligned;
2538 AllocationResult allocation = AllocateRaw(size, space, space, align);
2538 if (!allocation.To(&obj)) return allocation; 2539 if (!allocation.To(&obj)) return allocation;
2539 } 2540 }
2540 #ifdef DEBUG 2541 #ifdef DEBUG
2541 MemoryChunk* chunk = MemoryChunk::FromAddress(obj->address()); 2542 MemoryChunk* chunk = MemoryChunk::FromAddress(obj->address());
2542 DCHECK(chunk->owner()->identity() == space); 2543 DCHECK(chunk->owner()->identity() == space);
2543 #endif 2544 #endif
2544 CreateFillerObjectAt(obj->address(), size); 2545 CreateFillerObjectAt(obj->address(), size);
2545 return obj; 2546 return obj;
2546 } 2547 }
2547 2548
(...skipping 3981 matching lines...) Expand 10 before | Expand all | Expand 10 after
6529 } 6530 }
6530 delete list; 6531 delete list;
6531 } else { 6532 } else {
6532 prev = list; 6533 prev = list;
6533 } 6534 }
6534 list = next; 6535 list = next;
6535 } 6536 }
6536 } 6537 }
6537 } 6538 }
6538 } // namespace v8::internal 6539 } // namespace v8::internal
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698