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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to