[flang] Avoid copying scope. Improve readability.

Original-commit: flang-compiler/f18@5d2213cd7a68e9d125304e5d2e6e7e7fd8472b99
Tree-same-pre-rewrite: false

diff --git a/flang/lib/semantics/check-return.cc b/flang/lib/semantics/check-return.cc
index b280f13..19d4994 100644 (file)
--- a/flang/lib/semantics/check-return.cc
+++ b/flang/lib/semantics/check-return.cc
@@ -38,8 +38,8 @@ void ReturnStmtChecker::Leave(const parser::ReturnStmt &returnStmt) {
   // subroutine subprogram.
   // C1575 The scalar-int-expr is allowed only in the inclusive scope of a
   // subroutine subprogram.
-  auto scope{context_.FindScope(*context_.location())};
-  auto subprogramScope{FindContainingSubprogram(scope)};
+  const auto &scope{context_.FindScope(*context_.location())};
+  const auto &subprogramScope{FindContainingSubprogram(scope)};
   if (subprogramScope == nullptr) {
     context_.Say(
         "RETURN must in the inclusive scope of a SUBPROGRAM"_err_en_US);