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

Side by Side Diff: src/heap/spaces.h

Issue 1265443003: remove recursion from NewSpace::AllocateRaw* (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: address comments from the code review Created 5 years, 4 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 | src/heap/spaces.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2011 the V8 project authors. All rights reserved. 1 // Copyright 2011 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 #ifndef V8_HEAP_SPACES_H_ 5 #ifndef V8_HEAP_SPACES_H_
6 #define V8_HEAP_SPACES_H_ 6 #define V8_HEAP_SPACES_H_
7 7
8 #include "src/allocation.h" 8 #include "src/allocation.h"
9 #include "src/base/atomicops.h" 9 #include "src/base/atomicops.h"
10 #include "src/base/bits.h" 10 #include "src/base/bits.h"
(...skipping 2640 matching lines...) Expand 10 before | Expand all | Expand 10 after
2651 // to be lower than actual limit and then will gradually increase it 2651 // to be lower than actual limit and then will gradually increase it
2652 // in steps to guarantee that we do incremental marking steps even 2652 // in steps to guarantee that we do incremental marking steps even
2653 // when all allocation is performed from inlined generated code. 2653 // when all allocation is performed from inlined generated code.
2654 intptr_t inline_allocation_limit_step_; 2654 intptr_t inline_allocation_limit_step_;
2655 2655
2656 Address top_on_previous_step_; 2656 Address top_on_previous_step_;
2657 2657
2658 HistogramInfo* allocated_histogram_; 2658 HistogramInfo* allocated_histogram_;
2659 HistogramInfo* promoted_histogram_; 2659 HistogramInfo* promoted_histogram_;
2660 2660
2661 MUST_USE_RESULT AllocationResult 2661 bool EnsureAllocation(int size_in_bytes, AllocationAlignment alignment);
2662 SlowAllocateRaw(int size_in_bytes, AllocationAlignment alignment);
2663 2662
2664 friend class SemiSpaceIterator; 2663 friend class SemiSpaceIterator;
2665 }; 2664 };
2666 2665
2667 2666
2668 // ----------------------------------------------------------------------------- 2667 // -----------------------------------------------------------------------------
2669 // Old object space (includes the old space of objects and code space) 2668 // Old object space (includes the old space of objects and code space)
2670 2669
2671 class OldSpace : public PagedSpace { 2670 class OldSpace : public PagedSpace {
2672 public: 2671 public:
(...skipping 226 matching lines...) Expand 10 before | Expand all | Expand 10 after
2899 count = 0; 2898 count = 0;
2900 } 2899 }
2901 // Must be small, since an iteration is used for lookup. 2900 // Must be small, since an iteration is used for lookup.
2902 static const int kMaxComments = 64; 2901 static const int kMaxComments = 64;
2903 }; 2902 };
2904 #endif 2903 #endif
2905 } 2904 }
2906 } // namespace v8::internal 2905 } // namespace v8::internal
2907 2906
2908 #endif // V8_HEAP_SPACES_H_ 2907 #endif // V8_HEAP_SPACES_H_
OLDNEW
« no previous file with comments | « no previous file | src/heap/spaces.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698