[flang] Remove extraneous Pre/Post for ProcedureDeclarationStmt
authorTim Keith <tkeith@nvidia.com>
Wed, 13 Jun 2018 00:46:41 +0000 (17:46 -0700)
committerTim Keith <tkeith@nvidia.com>
Wed, 13 Jun 2018 00:46:41 +0000 (17:46 -0700)
commit5fcec63769826f5b698f79f3eae760f70bc7bc92
treef2a823b6e14bc24199c143eb70cabf340a4bcf75
parent3bccd5f0f53d94a1467adc5782c6d94c67061ad4
[flang] Remove extraneous Pre/Post for ProcedureDeclarationStmt

clang give a compilation error on resolve-names.cc because there are
two overloadings of Pre(ProcedureDeclarationStmt) available in
ResolveNamesVisitor. One is defined in DeclTypeSpecVisitor and the other
in DeclarationVisitor. They are both brought in to ResolveNamesVisitor
via `using` statements.

The one in DeclarationVisitor is the one that is supposed to be called.
The other should have been removed when this one was added. This is the
one that gcc chooses to call, so this doesn't change any behavior.

The same applies to the Post method as well.

Original-commit: flang-compiler/f18@872f8faf96164a01f18100fc1942f68523be6d76
Reviewed-on: https://github.com/flang-compiler/f18/pull/100
flang/lib/semantics/resolve-names.cc