[PATCH] D107154: [OpenCL] Add support of __opencl_c_pipes feature macro.
Anton Zabaznov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jul 30 06:25:13 PDT 2021
azabaznov added a comment.
> We might need to fix the target triple in clang/test/SemaOpenCL/invalid-pipes-cl2.0.cl to avoid surprises with testing but it has better coverage if we don't though...
Yeah, I am not sure now though. It's awful because testing is not deterministic as we have different default targets here, but on the other hand it helps to catch errors like this. Maybe it makes sense to change the existing target to spir and also add a random target here, such as nvptx for example?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D107154/new/
https://reviews.llvm.org/D107154
More information about the cfe-commits
mailing list