[PATCH] D118887: [OpenMP][Clang] Allow ancestor device modifier only with reverse offloading
Saiyedul Islam via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Feb 3 05:14:13 PST 2022
saiislam added inline comments.
================
Comment at: clang/test/OpenMP/target_device_codegen.cpp:43-47
- // CHECK-NOT: call i32 @__tgt_target_mapper(%struct.ident_t* @{{.+}},
- // CHECK: call void @__omp_offloading_{{.+}}_l46()
- // CHECK-NOT: call i32 @__tgt_target_mapper(%struct.ident_t* @{{.+}},
- #pragma omp target device(ancestor: n)
- ;
----------------
ABataev wrote:
> Do we have a codegen test for the ancestor modifier?
We shouldn't have a test for ancestor modifier because it can only be used when requires reverse_offload is specified and Spec 5.2 says that if an implementation is not supporting a requirement (reverse offload in this case) then it should give compile-time error termination [1].
I am going to propose this change in a different phab review.
[1] [[ https://www.openmp.org/wp-content/uploads/OpenMP-API-Specification-5-2.pdf | OpenMP API Specification 5.2 ]], Section 8.2.1, lines 12-13, page 212.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D118887/new/
https://reviews.llvm.org/D118887
More information about the cfe-commits
mailing list