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

Unified Diff: src/deoptimizer.h

Issue 2468183004: Used ZoneChunkList in deoptimizer (Closed)
Patch Set: Added unittests to ensure correct copyto Created 4 years, 1 month 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | src/deoptimizer.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/deoptimizer.h
diff --git a/src/deoptimizer.h b/src/deoptimizer.h
index 4fb7851710bb227fd60749c15a934e38682011e0..4d84fb76e879f005c6f11abfb971334f6ff5d13f 100644
--- a/src/deoptimizer.h
+++ b/src/deoptimizer.h
@@ -9,6 +9,7 @@
#include "src/deoptimize-reason.h"
#include "src/macro-assembler.h"
#include "src/source-position.h"
+#include "src/zone/zone-chunk-list.h"
namespace v8 {
namespace internal {
@@ -844,15 +845,15 @@ class DeoptimizerData {
class TranslationBuffer BASE_EMBEDDED {
public:
- explicit TranslationBuffer(Zone* zone) : contents_(256, zone) { }
+ explicit TranslationBuffer(Zone* zone) : contents_(zone) {}
- int CurrentIndex() const { return contents_.length(); }
- void Add(int32_t value, Zone* zone);
+ int CurrentIndex() const { return static_cast<int>(contents_.size()); }
+ void Add(int32_t value);
Handle<ByteArray> CreateByteArray(Factory* factory);
private:
- ZoneList<uint8_t> contents_;
+ ZoneChunkList<uint8_t> contents_;
};
@@ -917,9 +918,9 @@ class Translation BASE_EMBEDDED {
: buffer_(buffer),
index_(buffer->CurrentIndex()),
zone_(zone) {
- buffer_->Add(BEGIN, zone);
- buffer_->Add(frame_count, zone);
- buffer_->Add(jsframe_count, zone);
+ buffer_->Add(BEGIN);
+ buffer_->Add(frame_count);
+ buffer_->Add(jsframe_count);
}
int index() const { return index_; }
« no previous file with comments | « no previous file | src/deoptimizer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698