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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D150682: [clang]... Sergei Barannikov via Phabricator via cfe-commits

Reply via email to