[all-commits] [llvm/llvm-project] 681978: [flang] Duplicate names for ac-implied-do variable...

[Pete Steinfeld via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 681978d3b4e8fab98dd5c5c1f1687d3eff790a95
      https://github.com/llvm/llvm-project/commit/681978d3b4e8fab98dd5c5c1f1687d3eff790a95
  Author: Peter Steinfeld <psteinfeld at nvidia.com>
  Date:   2020-11-16 (Mon, 16 Nov 2020)

  Changed paths:
    M flang/lib/Semantics/resolve-names.cpp
    A flang/test/Evaluate/folding15.f90
    M flang/test/Semantics/array-constr-values.f90
    M flang/test/Semantics/symbol05.f90

  Log Message:
  -----------
  [flang] Duplicate names for ac-implied-do variables erroneously cause errors

According to section 19.4, paragraph 5, the scope of an ac-implied-do variable
is the enclosing ac-implied-do.  But we were not creating new scopes upon
entry to an ac-implied-do.  This was causing error messages to be erroneously
emitted.

I fixed, the code, added a test to array-constr-values.f90, added the test
folding15.f90 and corrected the test symbol05.f90.

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