[all-commits] [llvm/llvm-project] bf0870: [flang] Fix bug in IMPLICIT NONE(EXTERNAL)

[Tim Keith via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: bf0870d8640df27b6bb082de0347008240cb0e67
      https://github.com/llvm/llvm-project/commit/bf0870d8640df27b6bb082de0347008240cb0e67
  Author: Tim Keith <tkeith at nvidia.com>
  Date:   2020-12-18 (Fri, 18 Dec 2020)

  Changed paths:
    M flang/lib/Semantics/resolve-names.cpp
    M flang/test/Semantics/implicit07.f90

  Log Message:
  -----------
  [flang] Fix bug in IMPLICIT NONE(EXTERNAL)

We were only checking the restrictions of IMPLICIT NONE(EXTERNAL) when a
procedure name is first encountered. But it can also happen with an
existing symbol, e.g. if an external function's return type is declared
before is it called. This change adds a check in that branch too.

Differential Revision: https://reviews.llvm.org/D93552