[PATCH] D91560: [flang] Duplicate names for ac-implied-do variables erroneously cause errors

Mon Nov 16 14:42:50 PST 2020

klausler requested changes to this revision.
klausler added inline comments.
This revision now requires changes to proceed.

================
Comment at: flang/test/Evaluate/folding15.f90:7
+  integer :: dups(21) = &
+    [ ((iDuplicate, iDuplicate = 1,j), &
+       (2 * iDuplicate, iDuplicate = j,3 ), &
----------------
PeteSteinfeld wrote:
> klausler wrote:
> > Can you test using the "outer" iDuplicate as part of an implied DO loop bound expression for the "inner" iDuplicate, please?
> I'm not sure what you mean.  In the two tests I added, the "iDuplicate" variables are defined in parallel scopes.  It's not allowed to have an ac-implied-do variable with the same name as one in an outer scope.
> 
> Can you give me an example?
Actually, I can't, because I think that C7115 implies that the kind of scoping that you're implementing isn't necessary in the first place.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D91560/new/

https://reviews.llvm.org/D91560