================ @@ -361,9 +436,35 @@ mlir::LogicalResult CIRGenFunction::emitOMPTargetExitDataDirective( } mlir::LogicalResult CIRGenFunction::emitOMPTargetParallelDirective( ---------------- skatrak wrote:
I guess we're missing the combined construct dispatch and have to manually handle combined constructs per-leaf like this. Not a change request for this PR, but something to think about since the list of supported OpenMP combined constructs is pretty massive and this approach wouldn't scale. I think that should also make the implementation for `leafIsCombined` and `getLeafClauses` potentially more efficient, since we shouldn't have to decompose and search for the right leaf every time. https://github.com/llvm/llvm-project/pull/207019 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
