diff options
author | Razvan Lupusoru <razvan.lupusoru@gmail.com> | 2023-10-18 14:55:42 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-10-18 14:55:42 -0700 |
commit | 362b1157868c019e28ffca545a7d85c46c375ded (patch) | |
tree | 2d24c9e747c887af8a596b6f8c0ab37ba276942e /llvm/lib/Target/WebAssembly/WebAssemblyTargetMachine.cpp | |
parent | 077d89f0ee840d88a7d59a4f85d152d9c6f4758a (diff) | |
download | llvm-362b1157868c019e28ffca545a7d85c46c375ded.zip llvm-362b1157868c019e28ffca545a7d85c46c375ded.tar.gz llvm-362b1157868c019e28ffca545a7d85c46c375ded.tar.bz2 |
[flang][openacc] Avoid privatizing symbols during semantics (#69506)
During flang handling of semantics of OpenACC private/firstprivate/
reduction clauses (including the implicitly private loop IV), a new
scoped symbol was being created. This could lead to ambiguity in the
lowered FIR - aka having multiple fir.declare for the same symbol.
Because lowering of OpenACC does not materialize the meaning of the
private clauses (by actually creating a scoped local symbol), it does
not make sense to create a new symbol in semantics either.
I updated the acc-symbols01.f90 test to reflect this updated approach.
Technically, the test could be removed, but it made sense to keep in
place to highlight this intentional decision.
Diffstat (limited to 'llvm/lib/Target/WebAssembly/WebAssemblyTargetMachine.cpp')
0 files changed, 0 insertions, 0 deletions