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

Unified Diff: src/lithium.h

Issue 10966031: Lowered kMaxVirtualRegisters (fixes v8 2139 and chrome 123822 and 128252). (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 8 years, 3 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/lithium.h
diff --git a/src/lithium.h b/src/lithium.h
index e1cd52aa0acc4e9642c7c3f9ef14646de6290a04..fdc61cb6fef3a316c8555433166a690d99e70a5f 100644
--- a/src/lithium.h
+++ b/src/lithium.h
@@ -153,7 +153,10 @@ class LUnallocated: public LOperand {
kVirtualRegisterWidth> {
};
- static const int kMaxVirtualRegisters = 1 << kVirtualRegisterWidth;
+ // kMaxVirtualRegisters should not exceed 1 << kVirtualRegisterWidth but we
+ // are setting it to a lower value otherwise crankshaft is too slow.
+ STATIC_ASSERT(kVirtualRegisterWidth > 13);
+ static const int kMaxVirtualRegisters = 1 << 13;
Vyacheslav Egorov (Google) 2012/09/21 11:42:08 can you give some of this bits to other fields? i
static const int kMaxFixedIndex = 63;
static const int kMinFixedIndex = -64;
« 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