This is an archive of the discontinued LLVM Phabricator instance.

[clang] Convert a few OpenMP tests to opaque pointers
ClosedPublic

Authored by barannikov88 on May 16 2023, 8:27 AM.

Diff Detail

Event Timeline

barannikov88 created this revision.May 16 2023, 8:27 AM
barannikov88 requested review of this revision.May 16 2023, 8:27 AM
Herald added a project: Restricted Project. · View Herald Transcript

The changes in all tests are identical.

clang/test/OpenMP/target_depend_codegen.cpp
232

This GEP has disappeared (no-op, I guess), I changed it to load.

255–258

These, too. The results of the loads are now passed to the call above.

clang/test/OpenMP/target_parallel_depend_codegen.cpp
204

This one disappeared too, but since it doesn't capture the argument and the result is unused, I just removed it.

nikic accepted this revision.May 16 2023, 8:39 AM

LGTM

This revision is now accepted and ready to land.May 16 2023, 8:39 AM