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

[Johannes Doerfert via Phabricator via Openmp-commits]

jdoerfert added a comment.

In D102107#3639615 <https://reviews.llvm.org/D102107#3639615>, @dhruvachak wrote:

> In D102107#3639556 <https://reviews.llvm.org/D102107#3639556>, @jdoerfert wrote:
>
>> In D102107#3639551 <https://reviews.llvm.org/D102107#3639551>, @dhruvachak wrote:
>>
>>> Is there an llvm/utils script to update clang tests that have RUN lines at the top? An example is clang/test/OpenMP/debug_threadprivate_copyin.c.
>>
>> You can create the run lines with the `llvm/utils/update_cc_test_checks.py` script but those tests have manual lines for now.
>> I usually run `llvm/utils/update_cc_test_checks.py -u clang/test/OpenMP/*.{c,cpp}` to update all autogenerated tests.
>
> Okay, I will convert those few manual OpenMP tests to autogen format.
>
> How about the AST ones? Do they have to be manually updated? Example: clang/test/AST/ast-dump-openmp-distribute-parallel-for-simd.c

For these ones I have a script locally (attached) that need some manual doing but it helps:

  1. run the ast dump and store the result (same as RUN line), e.g.,
    {F23722650} clang -cc1 -internal-isystem /data/build/llvm-project/lib/clang/13.0.0/include -nostdsysteminc -triple x86_64-unknown-unknown -fopenmp -verify -ast-dump /data/src/llvm-project/clang/test/AST/ast-dump-openmp-begin-declare-variant_template_4.cpp &> /tmp/ast
  2. python3 ast_dump_2_check.py /tmp/ast CHECK
  3. replace the check lines with the content of /tmp/ast.check


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102107