[flang-commits] [flang] [Flang][OpenMP] DEFAULT(NONE) error checking on implicit references (PR #182214)

[Phoebe Linck via flang-commits]

phi-bee wrote:

It looks like `ResolveSeqLoopIndexInParallelOrTaskConstruct()` should be setting the loop index var private in the enclosing parallel in the way @dreachem describes, but it isn't behaving correctly. If it was, I think the error wouldn't be emitted for @sscalpone's snippet, since it would see that `i1` was privatized before being referenced.

So I think the solution is to fix `ResolveSeqLoopIndexInParallelOrTaskConstruct()` so it behaves as @dreachem describes the rule. I'll work on a patch for that unless there are more arguments for treating the case differently. If this PR needs to be reverted bc of the failing Fujitsu tests, I'll reintroduce it after that's fixed.

https://github.com/llvm/llvm-project/pull/182214