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

Unified Diff: runtime/vm/scavenger.cc

Issue 2146713004: More changes to use #ifndef PRODUCT ... #endif explicitly instead of relying on compiler magic. (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Created 4 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
« no previous file with comments | « runtime/vm/scavenger.h ('k') | runtime/vm/tags.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/scavenger.cc
diff --git a/runtime/vm/scavenger.cc b/runtime/vm/scavenger.cc
index 4eea99090cedf95d7a730dcec49575a7c4c1cd5c..cb028cee155eea76a20424b21849dd8614208eae 100644
--- a/runtime/vm/scavenger.cc
+++ b/runtime/vm/scavenger.cc
@@ -487,6 +487,7 @@ void Scavenger::IterateStoreBuffers(Isolate* isolate,
void Scavenger::IterateObjectIdTable(Isolate* isolate,
ScavengerVisitor* visitor) {
+#ifndef PRODUCT
if (!FLAG_support_service) {
return;
}
@@ -497,6 +498,7 @@ void Scavenger::IterateObjectIdTable(Isolate* isolate,
return;
}
ring->VisitPointers(visitor);
+#endif // !PRODUCT
}
@@ -858,6 +860,7 @@ void Scavenger::WriteProtect(bool read_only) {
}
+#ifndef PRODUCT
void Scavenger::PrintToJSONObject(JSONObject* object) const {
if (!FLAG_support_service) {
return;
@@ -885,6 +888,7 @@ void Scavenger::PrintToJSONObject(JSONObject* object) const {
space.AddProperty64("external", ExternalInWords() * kWordSize);
space.AddProperty("time", MicrosecondsToSeconds(gc_time_micros()));
}
+#endif // !PRODUCT
void Scavenger::AllocateExternal(intptr_t size) {
« no previous file with comments | « runtime/vm/scavenger.h ('k') | runtime/vm/tags.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698