[PATCH] D150682: [clang] Convert a few OpenMP tests to opaque pointers

[Sergei Barannikov via Phabricator via cfe-commits]

barannikov88 added a comment.

The changes in all tests are identical.



================
Comment at: clang/test/OpenMP/target_depend_codegen.cpp:232
-// CHECK:       store void (i8*, ...)* null, void (i8*, ...)** %
-// CHECK:       [[DEVICE_CAP:%.+]] = getelementptr inbounds %{{.+}}, %{{.+}}* %{{.+}}, i32 0, i32 0
-// CHECK:       [[DEV:%.+]] = load i32, i32* [[DEVICE_CAP]],
----------------
This GEP has disappeared (no-op, I guess), I changed it to load.



================
Comment at: clang/test/OpenMP/target_depend_codegen.cpp:255-258
-// CHECK:       [[BPR:%.+]] = getelementptr inbounds [3 x i8*], [3 x i8*]* [[BPTR_REF]], i[[SZ]] 0, i[[SZ]] 0
-// CHECK:       [[PR:%.+]] = getelementptr inbounds [3 x i8*], [3 x i8*]* [[PTR_REF]], i[[SZ]] 0, i[[SZ]] 0
-// CHECK:       [[SZT:%.+]] = getelementptr inbounds [3 x i64], [3 x i64]* [[SZ_REF]], i[[SZ]] 0, i[[SZ]] 0
-// CHECK:       [[M:%.+]] = getelementptr inbounds [3 x i8*], [3 x i8*]* [[M_REF]], i[[SZ]] 0, i[[SZ]] 0
----------------
These, too. The results of the loads are now passed to the call above.



================
Comment at: clang/test/OpenMP/target_parallel_depend_codegen.cpp:204
-// CHECK:       call void [[FN]](
-// CHECK:       [[SZT:%.+]] = getelementptr inbounds [2 x i64], [2 x i64]* %{{.+}}, i[[SZ]] 0, i[[SZ]] 0
-// CHECK:       [[DEVICE_CAP:%.+]] = getelementptr inbounds %{{.+}}, %{{.+}}* %{{.+}}, i32 0, i32 2
----------------
This one disappeared too, but since it doesn't capture the argument and the result is unused, I just removed it.



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150682