Conversation
| Decl* D = static_cast<Decl*>(scope); | ||
| if (auto *FD = llvm::dyn_cast<FunctionDecl>(D)) { | ||
| for (const auto &i: FD->attrs()) { | ||
| if (i->getKind() == clang::attr::Kind::CUDAGlobal) |
There was a problem hiding this comment.
warning: no header providing "clang::attr::Kind" is directly included [misc-include-cleaner]
lib/CppInterOp/CppInterOp.cpp:41:
- #if CLANG_VERSION_MAJOR >= 19
+ #include <clang/Basic/AttrKinds.h>
+ #if CLANG_VERSION_MAJOR >= 19
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #745 +/- ##
==========================================
- Coverage 79.04% 78.89% -0.15%
==========================================
Files 9 9
Lines 3879 3886 +7
==========================================
Hits 3066 3066
- Misses 813 820 +7
🚀 New features to boost your workflow:
|
29b9f9d to
6fe5ccc
Compare
|
@Vipul-Cariappa We don't currently have a ci job which can test cuda related stuff. The self hosted runners Clad uses for this are already over subscribed (CppInterOp technically has them available, but I never put them to use for this reason). If you add a test I can check your changes locally though. |
| CPPINTEROP_API bool IsClass(TCppScope_t scope); | ||
|
|
||
| /// Checks if the scope is a CUDA function or not. | ||
| CPPINTEROP_API bool IsCUDAFunction(TCppScope_t scope); |
There was a problem hiding this comment.
Can we generalize this api to be "HasAnnotation(AnnotationKind [ | AnnotationKind...] )"
Description
Fixes # (issue)
Type of change
Testing
Checklist