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

Unified Diff: src/assembler.h

Issue 1759873002: Assembler changes for enabling GrowHeap in Wasm (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Adding x64 test Created 4 years, 10 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 side-by-side diff with in-line comments
Download patch
Index: src/assembler.h
diff --git a/src/assembler.h b/src/assembler.h
index 2224b9fb12e2363e681bf3c0b711a717ad226929..00e9d93e805ebfe77f1b2eee322b4c9273697dcc 100644
--- a/src/assembler.h
+++ b/src/assembler.h
@@ -385,6 +385,9 @@ class RelocInfo {
DEBUGGER_STATEMENT, // Code target for the debugger statement.
EMBEDDED_OBJECT,
CELL,
+ // Marker for relocation when the heap changes, used only on wasm code
+ // objects
Yang 2016/03/03 06:21:56 Can you change the comment to say "To relocate poi
gdeepti1 2016/03/03 18:00:10 Done.
+ WASM_HEAP_OBJECT,
titzer 2016/03/03 18:11:59 Can we name this WASM_MEMORY_REFERENCE? We've bee
gdeepti1 2016/03/03 22:59:15 Done.
// Everything after runtime_entry (inclusive) is not GC'ed.
RUNTIME_ENTRY,
@@ -427,7 +430,7 @@ class RelocInfo {
FIRST_REAL_RELOC_MODE = CODE_TARGET,
LAST_REAL_RELOC_MODE = VENEER_POOL,
LAST_CODE_ENUM = DEBUGGER_STATEMENT,
- LAST_GCED_ENUM = CELL,
+ LAST_GCED_ENUM = WASM_HEAP_OBJECT,
};
STATIC_ASSERT(NUMBER_OF_MODES <= kBitsPerInt);
@@ -511,6 +514,9 @@ class RelocInfo {
static inline bool IsGeneratorContinuation(Mode mode) {
return mode == GENERATOR_CONTINUATION;
}
+ static inline bool IsWasmCodeEntry(Mode mode) {
Yang 2016/03/03 06:21:56 This name is misleading. I would have thought that
gdeepti1 2016/03/03 18:00:10 Name was left over from a previous RelocInfo mode
titzer 2016/03/03 18:11:59 As above, IsWasmMemoryReference, for consistency.
+ return mode == WASM_HEAP_OBJECT;
+ }
static inline int ModeMask(Mode mode) { return 1 << mode; }
// Accessors

Powered by Google App Engine
This is Rietveld 408576698