Only left IsAlternateEntry() with pointer arg

Author: eugeneepshteyn

flang/include/flang/Evaluate/tools.h

@@ -1536,9 +1536,6 @@ inline bool IsAlternateEntry(const Symbol *symbol) {
   // symbol.
   return symbol && GetMainEntry(symbol) != symbol;
 }
-inline bool IsAlternateEntry(const Symbol &symbol) {
-  return IsAlternateEntry(&symbol);
-}
 
 // These functions are used in Evaluate so they are defined here rather than in
 // Semantics to avoid a link-time dependency on Semantics.

flang/lib/Semantics/check-declarations.cpp

@@ -2950,7 +2950,7 @@ static bool IsSubprogramDefinition(const Symbol &symbol) {
 
 static bool IsExternalProcedureDefinition(const Symbol &symbol) {
   return IsBlockData(symbol) ||
-      ((IsSubprogramDefinition(symbol) || IsAlternateEntry(symbol)) &&
+      ((IsSubprogramDefinition(symbol) || IsAlternateEntry(&symbol)) &&
           (IsExternal(symbol) || symbol.GetBindName()));
 }