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

Unified Diff: src/compiler/bytecode-analysis.h

Issue 2523893003: Reland of [ignition/turbo] Perform liveness analysis on the bytecodes (Closed)
Patch Set: Export handler table for tests 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 | « src/DEPS ('k') | src/compiler/bytecode-analysis.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/compiler/bytecode-analysis.h
diff --git a/src/compiler/bytecode-analysis.h b/src/compiler/bytecode-analysis.h
index f50b00f2c696ecd0acf07ef2d8c7276267555df4..9baba84f92bbf1f92520f5c189d8f5d6fa527499 100644
--- a/src/compiler/bytecode-analysis.h
+++ b/src/compiler/bytecode-analysis.h
@@ -5,6 +5,9 @@
#ifndef V8_COMPILER_BYTECODE_ANALYSIS_H_
#define V8_COMPILER_BYTECODE_ANALYSIS_H_
+#include "src/base/hashmap.h"
+#include "src/bit-vector.h"
+#include "src/compiler/bytecode-liveness-map.h"
#include "src/handles.h"
#include "src/zone/zone-containers.h"
@@ -15,9 +18,10 @@ class BytecodeArray;
namespace compiler {
-class BytecodeAnalysis BASE_EMBEDDED {
+class V8_EXPORT_PRIVATE BytecodeAnalysis BASE_EMBEDDED {
public:
- BytecodeAnalysis(Handle<BytecodeArray> bytecode_array, Zone* zone);
+ BytecodeAnalysis(Handle<BytecodeArray> bytecode_array, Zone* zone,
+ bool do_liveness_analysis);
// Analyze the bytecodes to find the loop ranges and nesting. No other
// methods in this class return valid information until this has been called.
@@ -32,13 +36,33 @@ class BytecodeAnalysis BASE_EMBEDDED {
// at {header_offset}, or -1 for outer-most loops.
int GetParentLoopFor(int header_offset) const;
+ // Gets the in-liveness for the bytecode at {offset}. The liveness bit vector
+ // represents the liveness of the registers and the accumulator, with the last
+ // bit being the accumulator liveness bit, and so is (register count + 1) bits
+ // long.
+ const BitVector* GetInLivenessFor(int offset) const;
+
+ // Gets the out-liveness for the bytecode at {offset}. The liveness bit vector
+ // represents the liveness of the registers and the accumulator, with the last
+ // bit being the accumulator liveness bit, and so is (register count + 1) bits
+ // long.
+ const BitVector* GetOutLivenessFor(int offset) const;
+
+ std::ostream& PrintLivenessTo(std::ostream& os) const;
+
private:
void PushLoop(int loop_header, int loop_end);
+#if DEBUG
+ bool LivenessIsValid();
+#endif
+
Zone* zone() const { return zone_; }
Handle<BytecodeArray> bytecode_array() const { return bytecode_array_; }
+ private:
Handle<BytecodeArray> bytecode_array_;
+ bool do_liveness_analysis_;
Zone* zone_;
ZoneStack<int> loop_stack_;
@@ -46,6 +70,8 @@ class BytecodeAnalysis BASE_EMBEDDED {
ZoneMap<int, int> end_to_header_;
ZoneMap<int, int> header_to_parent_;
+ BytecodeLivenessMap liveness_map_;
+
DISALLOW_COPY_AND_ASSIGN(BytecodeAnalysis);
};
« no previous file with comments | « src/DEPS ('k') | src/compiler/bytecode-analysis.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698