Mordante created this revision.
Mordante added reviewers: rsmith, rtrieu, aaron.ballman, xbolva00.
Mordante added a project: clang.
When Wrange-loop-analysis issues a diagnostic on a dependent type in a template
the diagnostic may not be valid for all instantiations. Therefore the
diagnostic is suppressed during the instantiation. Non dependent types still
issue a diagnostic.
The same can happen when using macros. Therefore the diagnostic is disabled for
macros.
Fixes https://bugs.llvm.org/show_bug.cgi?id=44556
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D73007
Files:
clang/lib/Sema/SemaStmt.cpp
clang/test/SemaCXX/warn-range-loop-analysis.cpp
Index: clang/test/SemaCXX/warn-range-loop-analysis.cpp
===================================================================
--- clang/test/SemaCXX/warn-range-loop-analysis.cpp
+++ clang/test/SemaCXX/warn-range-loop-analysis.cpp
@@ -454,3 +454,75 @@
// expected-note@-2 {{'Bar'}}
// CHECK: fix-it:"{{.*}}":{[[@LINE-3]]:17-[[@LINE-3]]:18}:" "
}
+
+template <class T>
+void test_template_function() {
+ // In a template instantiation the diagnostics should not be emitted for
+ // loops with dependent types.
+ Container<Bar> C;
+ for (const Bar &x : C) {}
+ // expected-warning@-1 {{always a copy}}
+ // expected-note@-2 {{'Bar'}}
+ // CHECK: fix-it:"{{.*}}":{[[@LINE-3]]:18-[[@LINE-3]]:19}:""
+
+ Container<T> Dependent;
+ for (const T &x : Dependent) {}
+}
+template void test_template_function<Bar>();
+
+template <class T>
+struct test_template_struct {
+ static void static_member() {
+ Container<Bar> C;
+ for (const Bar &x : C) {}
+ // expected-warning@-1 {{always a copy}}
+ // expected-note@-2 {{'Bar'}}
+ // CHECK: fix-it:"{{.*}}":{[[@LINE-3]]:20-[[@LINE-3]]:21}:""
+
+ Container<T> Dependent;
+ for (const T &x : Dependent) {}
+ }
+
+ void member() {
+ Container<Bar> C;
+ for (const Bar &x : C) {}
+ // expected-warning@-1 {{always a copy}}
+ // expected-note@-2 {{'Bar'}}
+ // CHECK: fix-it:"{{.*}}":{[[@LINE-3]]:20-[[@LINE-3]]:21}:""
+
+ Container<T> Dependent;
+ for (const T &x : Dependent) {}
+ }
+};
+template struct test_template_struct<Bar>;
+
+struct test_struct_with_templated_member {
+ void member() {
+ Container<Bar> C;
+ for (const Bar &x : C) {}
+ // expected-warning@-1 {{always a copy}}
+ // expected-note@-2 {{'Bar'}}
+ // CHECK: fix-it:"{{.*}}":{[[@LINE-3]]:20-[[@LINE-3]]:21}:""
+ }
+
+ template <class T>
+ void template_member() {
+ Container<Bar> C;
+ for (const Bar &x : C) {}
+ // expected-warning@-1 {{always a copy}}
+ // expected-note@-2 {{'Bar'}}
+ // CHECK: fix-it:"{{.*}}":{[[@LINE-3]]:20-[[@LINE-3]]:21}:""
+
+ Container<T> Dependent;
+ for (const T &x : Dependent) {}
+ }
+};
+template void test_struct_with_templated_member::template_member<Bar>();
+
+#define TEST_MACRO \
+ void test_macro() { \
+ Container<Bar> C; \
+ for (const Bar &x : C) {} \
+ }
+
+TEST_MACRO
Index: clang/lib/Sema/SemaStmt.cpp
===================================================================
--- clang/lib/Sema/SemaStmt.cpp
+++ clang/lib/Sema/SemaStmt.cpp
@@ -2838,6 +2838,9 @@
/// Suggest "const foo &x" to prevent the copy.
static void DiagnoseForRangeVariableCopies(Sema &SemaRef,
const CXXForRangeStmt *ForStmt) {
+ if (SemaRef.inTemplateInstantiation())
+ return;
+
if (SemaRef.Diags.isIgnored(diag::warn_for_range_const_reference_copy,
ForStmt->getBeginLoc()) &&
SemaRef.Diags.isIgnored(diag::warn_for_range_variable_always_copy,
@@ -2860,6 +2863,9 @@
if (!InitExpr)
return;
+ if (InitExpr->getExprLoc().isMacroID())
+ return;
+
if (VariableType->isReferenceType()) {
DiagnoseForRangeReferenceVariableCopies(SemaRef, VD,
ForStmt->getRangeInit()->getType());
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
