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

Unified Diff: tools/clang/blink_gc_plugin/DiagnosticsReporter.cpp

Issue 2323463002: [blink_gc_plugin] Complain about stack-allocated classes with GC bases, but not arbitrary non-stack… (Closed)
Patch Set: Created 4 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
Index: tools/clang/blink_gc_plugin/DiagnosticsReporter.cpp
diff --git a/tools/clang/blink_gc_plugin/DiagnosticsReporter.cpp b/tools/clang/blink_gc_plugin/DiagnosticsReporter.cpp
index ef3a37d592a39c0f41ce1cfdb56fc89b3ad4c51d..bd46f2c78745940e2c13677b0cda71cb5b294c2c 100644
--- a/tools/clang/blink_gc_plugin/DiagnosticsReporter.cpp
+++ b/tools/clang/blink_gc_plugin/DiagnosticsReporter.cpp
@@ -127,9 +127,9 @@ const char kFieldRequiresFinalizationNote[] =
const char kManualDispatchMethodNote[] =
"[blink-gc] Manual dispatch %0 declared here:";
-const char kDerivesNonStackAllocated[] =
+const char kStackAllocatedDerivesGarbageCollected[] =
"[blink-gc] Stack-allocated class %0 derives class %1"
- " which is not stack allocated.";
+ " which is garbage collected.";
const char kClassOverridesNew[] =
"[blink-gc] Garbage collected class %0"
@@ -197,8 +197,8 @@ DiagnosticsReporter::DiagnosticsReporter(
diagnostic_.getCustomDiagID(getErrorLevel(), kMissingTraceDispatch);
diag_missing_finalize_dispatch_ =
diagnostic_.getCustomDiagID(getErrorLevel(), kMissingFinalizeDispatch);
- diag_derives_non_stack_allocated_ =
- diagnostic_.getCustomDiagID(getErrorLevel(), kDerivesNonStackAllocated);
+ diag_stack_allocated_derives_gc_ = diagnostic_.getCustomDiagID(
+ getErrorLevel(), kStackAllocatedDerivesGarbageCollected);
diag_class_overrides_new_ =
diagnostic_.getCustomDiagID(getErrorLevel(), kClassOverridesNew);
diag_class_declares_pure_virtual_trace_ = diagnostic_.getCustomDiagID(
@@ -453,11 +453,11 @@ void DiagnosticsReporter::ReportMissingDispatch(
ReportDiagnostic(dispatch->getLocStart(), error) << receiver->record();
}
-void DiagnosticsReporter::DerivesNonStackAllocated(
+void DiagnosticsReporter::StackAllocatedDerivesGarbageCollected(
RecordInfo* info,
BasePoint* base) {
ReportDiagnostic(base->spec().getLocStart(),
- diag_derives_non_stack_allocated_)
+ diag_stack_allocated_derives_gc_)
<< info->record() << base->info()->record();
}
« no previous file with comments | « tools/clang/blink_gc_plugin/DiagnosticsReporter.h ('k') | tools/clang/blink_gc_plugin/tests/stack_allocated.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698