[flang-commits] [flang] [flang][openmp] Support importing module declare target globals (PR #213930)

[Ville-Markus Yli-Suutala via flang-commits]

VeeEM wrote:

@ravurvi20 Thanks, I added a `-fopenmp-is-target-device` test.

My initial approach was pretty much the same as yours, to query the ultimate symbol for the attributes, then I switched to walking the module parse trees for the directives instead. I do think the current implementation is better because as you mentioned, it gets the functions too, and also it is very similar to how declare target directives that were not imported from a module file are handled.

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