Review usage of auto in Lower/ComponentPath.cpp (#1312)

clementval · web-flow · commit 57d743837f2d · 2021-12-07T19:27:03.000+01:00
diff --git a/flang/lib/Lower/ComponentPath.cpp b/flang/lib/Lower/ComponentPath.cpp
@@ -35,7 +35,7 @@ void Fortran::lower::ComponentPath::clear() {
 }
 
 bool Fortran::lower::isRankedArrayAccess(const Fortran::evaluate::ArrayRef &x) {
-  for (auto &sub : x.subscript()) {
+  for (const Fortran::evaluate::Subscript &sub : x.subscript()) {
     if (std::visit(
             Fortran::common::visitors{
                 [&](const Fortran::evaluate::Triplet &) { return true; },

Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ void Fortran::lower::ComponentPath::clear() {`
`35`	`35`	`}`
`36`	`36`
`37`	`37`	`bool Fortran::lower::isRankedArrayAccess(const Fortran::evaluate::ArrayRef &x) {`
`38`		`- for (auto &sub : x.subscript()) {`
	`38`	`+ for (const Fortran::evaluate::Subscript &sub : x.subscript()) {`
`39`	`39`	`if (std::visit(`
`40`	`40`	`Fortran::common::visitors{`
`41`	`41`	`[&](const Fortran::evaluate::Triplet &) { return true; },`