[Openmp-commits] [PATCH] D102107: [OpenMP] Codegen aggregate for outlined function captures

[Johannes Doerfert via Phabricator via Openmp-commits]

jdoerfert added a comment.

It seems the buildbot didn't actually test this patch but an old one, still:

The checks for this tests are not updated:
 target_teams_distribute_parallel_for_order_codegen.cpp
 target_in_reduction_codegen.cpp
 nvptx_lambda_capturing.cpp
 nvptx_lambda_pointer_capturing.cpp

Similar to clang/test/OpenMP/declare_variant_construct_codegen_1.c, we should manually update the few fork calls in clang/test/OpenMP/declare_variant_construct_codegen_1.c.

Can you share the output of the AST dump tests and the new check lines, so what run produces and the file we give to Filechec to verify it.



================
Comment at: clang/test/OpenMP/declare_variant_construct_codegen_1.c:1019
+//
+//
----------------
Something went wrong here. Might be easier to manually change the kmpc_forc_call line (should not be much more)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D102107/new/

https://reviews.llvm.org/D102107