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

Unified Diff: runtime/vm/intermediate_language_x64.cc

Issue 10800037: New linear scan allocator. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: refactored liveness computation Created 8 years, 5 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: runtime/vm/intermediate_language_x64.cc
diff --git a/runtime/vm/intermediate_language_x64.cc b/runtime/vm/intermediate_language_x64.cc
index 1db3c598dacc6ee128ae69dd2376b0d4570223f4..b50d4c524f93425150ebc8fb6e0cbfd8e29af7b6 100644
--- a/runtime/vm/intermediate_language_x64.cc
+++ b/runtime/vm/intermediate_language_x64.cc
@@ -501,8 +501,7 @@ LocationSummary* RelationalOpComp::MakeLocationSummary() const {
const intptr_t kNumInputs = 2;
const intptr_t kNumTemps = 1;
LocationSummary* summary = new LocationSummary(kNumInputs,
- kNumTemps,
- LocationSummary::kCall);
+ kNumTemps);
summary->set_in(0, Location::RequiresRegister());
summary->set_in(1, Location::RequiresRegister());
summary->set_out(Location::RequiresRegister());
@@ -1456,7 +1455,7 @@ LocationSummary* CheckStackOverflowComp::MakeLocationSummary() const {
LocationSummary* summary = new LocationSummary(kNumInputs,
kNumTemps,
LocationSummary::kCall);
- summary->set_temp(0, Location::RequiresRegister());
+ summary->set_temp(0, Location::RegisterLocation(R10));
srdjan 2012/07/22 15:09:24 Why R10?
Vyacheslav Egorov (Google) 2012/07/24 12:26:42 It should be any fixed register. Replaced with RAX
return summary;
}

Powered by Google App Engine
This is Rietveld 408576698