Adds extra error diagnostics when using unsupported types in kernel arguments.
This fixes https://bugs.llvm.org/show_bug.cgi?id=48099
Paths
| Differential D100471
[C++4OpenCL] Add extra diagnostics for kernel argument types ClosedPublic Authored by olestrohm on Apr 14 2021, 5:13 AM.
Details Summary Adds extra error diagnostics when using unsupported types in kernel arguments. This fixes https://bugs.llvm.org/show_bug.cgi?id=48099
Diff Detail Event Timeline
Comment Actions Added more exhaustive tests, as well as fixed the diagnostic to allow reference types.
Comment Actions LGTM from my side. I will leave final approval to Sven.
Comment Actions Restricted the checks to C++ for OpenCL and added another test for OpenCL vector types. This revision is now accepted and ready to land.Apr 16 2021, 5:47 AM Closed by commit rG362958ac7346: [C++4OpenCL] Add extra diagnostics for kernel argument types (authored by Anastasia). · Explain WhyApr 22 2021, 7:29 AM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 338032 clang/lib/Sema/SemaDecl.cpp
clang/test/SemaOpenCLCXX/invalid-kernel.clcpp
|
I am wondering if this should be made conditional on C++ mode? Or is there no possible way that this new return InvalidKernelParam can be triggered from OpenCL C mode?