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

Unified Diff: runtime/vm/cha.cc

Issue 2974233002: VM: Re-format to use at most one newline between functions (Closed)
Patch Set: Rebase and merge Created 3 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/branch_optimizer.cc ('k') | runtime/vm/cha_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/cha.cc
diff --git a/runtime/vm/cha.cc b/runtime/vm/cha.cc
index 83a2bc50b4dc71bff8fb89eca8c0958462cf885f..7045001c634fb65faf8fcfb347ae86f023401ccd 100644
--- a/runtime/vm/cha.cc
+++ b/runtime/vm/cha.cc
@@ -24,7 +24,6 @@ void CHA::AddToGuardedClasses(const Class& cls, intptr_t subclass_count) {
return;
}
-
bool CHA::IsGuardedClass(intptr_t cid) const {
for (intptr_t i = 0; i < guarded_classes_.length(); ++i) {
if (guarded_classes_[i].cls->id() == cid) return true;
@@ -32,7 +31,6 @@ bool CHA::IsGuardedClass(intptr_t cid) const {
return false;
}
-
bool CHA::HasSubclasses(const Class& cls) {
ASSERT(!cls.IsNull());
ASSERT(cls.id() >= kInstanceCid);
@@ -51,14 +49,12 @@ bool CHA::HasSubclasses(const Class& cls) {
return !direct_subclasses.IsNull() && (direct_subclasses.Length() > 0);
}
-
bool CHA::HasSubclasses(intptr_t cid) const {
const ClassTable& class_table = *thread_->isolate()->class_table();
Class& cls = Class::Handle(thread_->zone(), class_table.At(cid));
return HasSubclasses(cls);
}
-
bool CHA::ConcreteSubclasses(const Class& cls,
GrowableArray<intptr_t>* class_ids) {
if (cls.InVMHeap()) return false;
@@ -83,7 +79,6 @@ bool CHA::ConcreteSubclasses(const Class& cls,
return true;
}
-
bool CHA::IsImplemented(const Class& cls) {
// Function type aliases have different type checking rules.
ASSERT(!cls.IsTypedefClass());
@@ -96,7 +91,6 @@ bool CHA::IsImplemented(const Class& cls) {
return cls.is_implemented();
}
-
static intptr_t CountFinalizedSubclasses(Thread* thread, const Class& cls) {
intptr_t count = 0;
const GrowableObjectArray& cls_direct_subclasses =
@@ -116,7 +110,6 @@ static intptr_t CountFinalizedSubclasses(Thread* thread, const Class& cls) {
return count;
}
-
bool CHA::IsConsistentWithCurrentHierarchy() const {
for (intptr_t i = 0; i < guarded_classes_.length(); i++) {
const intptr_t subclass_count =
@@ -128,7 +121,6 @@ bool CHA::IsConsistentWithCurrentHierarchy() const {
return true;
}
-
bool CHA::HasOverride(const Class& cls,
const String& function_name,
intptr_t* subclasses_count) {
@@ -173,12 +165,10 @@ bool CHA::HasOverride(const Class& cls,
return false;
}
-
void CHA::RegisterDependencies(const Code& code) const {
for (intptr_t i = 0; i < guarded_classes_.length(); ++i) {
guarded_classes_[i].cls->RegisterCHACode(code);
}
}
-
} // namespace dart
« no previous file with comments | « runtime/vm/branch_optimizer.cc ('k') | runtime/vm/cha_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698