[clang] [llvm] [Clang][OpenMP] Support for dispatch construct (Sema & Codegen) support (PR #117904)

Alexey Bataev via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 12 12:10:59 PST 2025


alexey-bataev wrote:

> > #if dispatch transformed to
> > #pragma omp taskwait depend(out:x)
> > #pragma omp dispatch nocontext(c2)
> > foo(); --> with traits call to foo_variant_dispatch(i,j)
> > you should have 2 captured regions - one for taskwait and one for dispatch. It does not mean, >you need to transform the AST node, there should be single AST node with 2 captured regions >(one caotures another), just like it is done for combined constructs.
> 
> I do not have this code anymore in SemaOpenMP.cpp. I am having code for adding AnnotateAttr. In CodeGen this exact comment has been taken care of.

I'm saying how it should be implemented. Otherwise, it may crash in some cases or work incorrectly.

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


More information about the cfe-commits mailing list