diff options
| author | Tom Eccles <tom.eccles@arm.com> | 2025-05-13 10:14:02 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-05-13 10:14:02 +0100 |
| commit | 8ecb958b8f7bc8110fd2bd3e9b023095e7f14c94 (patch) | |
| tree | cb8964511ee66b41eb621db17276364beeecbde7 /clang/lib/CodeGen/ModuleBuilder.cpp | |
| parent | 83ce8a44bb7ec52a20fea1f2eb28c7756f46786b (diff) | |
| download | llvm-8ecb958b8f7bc8110fd2bd3e9b023095e7f14c94.zip llvm-8ecb958b8f7bc8110fd2bd3e9b023095e7f14c94.tar.gz llvm-8ecb958b8f7bc8110fd2bd3e9b023095e7f14c94.tar.bz2 | |
[flang][OpenMP][Semantics] resolve objects in the flush arg list (#139522)
Fixes #136583
Normally the flush argument list would contain a DataRef to some
variable. All DataRefs are handled generically in resolve-names and so
the problem wasn't observed. But when a common block name is specified,
this is not parsed as a DataRef. There was already handling in
resolve-directives for OmpObjectList but not for argument lists. I've
added a visitor for FLUSH which ensures all of the arguments have been
resolved.
The test is there to make sure the compiler doesn't crashed encountering
the unresolved symbol. It shows that we currently deny flushing a common
block. I'm not sure that it is right to restrict common blocks from
flush argument lists, but fixing that can come in a different patch.
This one is fixing an ICE.
Diffstat (limited to 'clang/lib/CodeGen/ModuleBuilder.cpp')
0 files changed, 0 insertions, 0 deletions