Fix review comments

Author: Leporacanthicus

flang/include/flang/Semantics/semantics.h

@@ -348,7 +348,7 @@ class SemanticsContext {
   UnorderedSymbolSet isDefined_;
   std::list<ProgramTree> programTrees_;
 
-  // storage for mangled names used in OMP DECLARE REDUCTION.
+  // Storage for mangled names used in OMP DECLARE REDUCTION.
   // use std::list to avoid re-allocating the string when adding
   // more content to the container.
   std::list<std::string> userReductionNames_;

flang/lib/Semantics/check-omp-structure.cpp

@@ -3498,7 +3498,7 @@ void OmpStructureChecker::CheckReductionObjects(
 
 static bool CheckSymbolSupportsType(const Scope &scope,
     const parser::CharBlock &name, const DeclTypeSpec &type) {
-  if (const auto &symbol{scope.FindSymbol(name)}) {
+  if (const auto *symbol{scope.FindSymbol(name)}) {
     if (const auto *reductionDetails{
             symbol->detailsIf<UserReductionDetails>()}) {
       return reductionDetails->SupportsType(type);

flang/lib/Semantics/mod-file.cpp

@@ -8,7 +8,6 @@
 
 #include "mod-file.h"
 #include "resolve-names.h"
-#include "flang/Common/indirection.h"
 #include "flang/Common/restorer.h"
 #include "flang/Evaluate/tools.h"
 #include "flang/Parser/message.h"

flang/lib/Semantics/resolve-names.cpp

@@ -1842,7 +1842,7 @@ void OmpVisitor::ProcessReductionSpecifier(
     reductionDetails = symbol->detailsIf<UserReductionDetails>();
 
     if (!reductionDetails) {
-      context().Say(name->source,
+      context().Say(
           "Duplicate definition of '%s' in DECLARE REDUCTION"_err_en_US,
           name->source);
       return;