[PATCH] D150733: [clang] Convert remaining OpenMP tests to opaque pointers

2023-05-17 Thread Sergei Barannikov via Phabricator via cfe-commits 
This revision was automatically updated to reflect the committed changes.
Closed by commit rG8be4bd16ca41: [clang] Convert remaining OpenMP tests to 
opaque pointers (authored by barannikov88).

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150733

Files:
  clang/test/OpenMP/declare_reduction_codegen.cpp
  clang/test/OpenMP/parallel_for_simd_codegen.cpp
  clang/test/OpenMP/simd_codegen.cpp
  clang/test/OpenMP/target_codegen.cpp
  clang/test/OpenMP/target_data_codegen.cpp
  clang/test/OpenMP/target_enter_data_codegen.cpp
  clang/test/OpenMP/target_enter_data_depend_codegen.cpp
  clang/test/OpenMP/target_exit_data_codegen.cpp
  clang/test/OpenMP/target_exit_data_depend_codegen.cpp
  clang/test/OpenMP/target_in_reduction_codegen.cpp
  clang/test/OpenMP/target_private_codegen.cpp
  clang/test/OpenMP/target_reduction_codegen.cpp
  clang/test/OpenMP/target_simd_codegen.cpp
  clang/test/OpenMP/target_update_depend_codegen.cpp

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

[PATCH] D150733: [clang] Convert remaining OpenMP tests to opaque pointers

2023-05-17 Thread Nikita Popov via Phabricator via cfe-commits 
nikic accepted this revision.
nikic added a comment.
This revision is now accepted and ready to land.

LGTM


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150733

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

[PATCH] D150733: [clang] Convert remaining OpenMP tests to opaque pointers

2023-05-16 Thread Sergei Barannikov via Phabricator via cfe-commits 
barannikov88 added a comment.

These are the last tests in OpenMP subdirectory.




Comment at: clang/test/OpenMP/target_simd_codegen.cpp:143
 
-  // CHECK:   call void [[HVT1:@.+]](i[[SZ]] {{[^,]+}}, i{{32|64}}{{[*]*}} 
{{[^)]+}})
+  // CHECK:   call void [[HVT1:@.+]](i[[SZ]] {{[^,]+}}, {{[^)]+}})
   long long k = get_val();

Depending on the RUN line, the second argument should match either ptr or an 
integer.
I couldn't invent a better solution than just to remove the check. Hopefully 
this is ok.



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150733

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

[PATCH] D150733: [clang] Convert remaining OpenMP tests to opaque pointers

2023-05-16 Thread Sergei Barannikov via Phabricator via cfe-commits 
barannikov88 created this revision.
barannikov88 added a reviewer: nikic.
Herald added subscribers: sunshaoce, StephenFan, guansong, yaxunl.
Herald added a project: All.
barannikov88 requested review of this revision.
Herald added a reviewer: jdoerfert.
Herald added subscribers: cfe-commits, jplehr, sstefan1.
Herald added a project: clang.

Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D150733

Files:
  clang/test/OpenMP/declare_reduction_codegen.cpp
  clang/test/OpenMP/parallel_for_simd_codegen.cpp
  clang/test/OpenMP/simd_codegen.cpp
  clang/test/OpenMP/target_codegen.cpp
  clang/test/OpenMP/target_data_codegen.cpp
  clang/test/OpenMP/target_enter_data_codegen.cpp
  clang/test/OpenMP/target_enter_data_depend_codegen.cpp
  clang/test/OpenMP/target_exit_data_codegen.cpp
  clang/test/OpenMP/target_exit_data_depend_codegen.cpp
  clang/test/OpenMP/target_in_reduction_codegen.cpp
  clang/test/OpenMP/target_private_codegen.cpp
  clang/test/OpenMP/target_reduction_codegen.cpp
  clang/test/OpenMP/target_simd_codegen.cpp
  clang/test/OpenMP/target_update_depend_codegen.cpp

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