Skip to content

Commit a84caa3

Browse files
committed
Added and edited comments.
1 parent 3b1ba7c commit a84caa3

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

Diff for: clang/lib/Sema/SemaSYCL.cpp

+3-2
Original file line numberDiff line numberDiff line change
@@ -306,14 +306,15 @@ static int64_t getIntExprValue(const Expr *E, ASTContext &Ctx) {
306306
return E->getIntegerConstantExpr(Ctx)->getSExtValue();
307307
}
308308

309-
// Collect function attributes related to SYCL
309+
// Collect function attributes related to SYCL.
310310
static void collectSYCLAttributes(Sema &S, FunctionDecl *FD,
311311
llvm::SmallVector<Attr *, 4> &Attrs,
312312
bool DirectlyCalled = true) {
313313
if (!FD->hasAttrs())
314314
return;
315315

316316
llvm::copy_if(FD->getAttrs(), std::back_inserter(Attrs), [](Attr *A) {
317+
// FIXME: Make this list self-adapt as new SYCL attributes are added.
317318
return isa<IntelReqdSubGroupSizeAttr, ReqdWorkGroupSizeAttr,
318319
SYCLIntelKernelArgsRestrictAttr, SYCLIntelNumSimdWorkItemsAttr,
319320
SYCLIntelSchedulerTargetFmaxMhzAttr,
@@ -3150,7 +3151,7 @@ void Sema::CheckSYCLKernelCall(FunctionDecl *KernelFunc, SourceRange CallLoc,
31503151
// For a wrapped parallel_for, copy attributes from original
31513152
// kernel to wrapped kernel.
31523153
void Sema::copySYCLKernelAttrs(const CXXRecordDecl *KernelObj) {
3153-
// Get the operator() function of the wrapper
3154+
// Get the operator() function of the wrapper.
31543155
CXXMethodDecl *OpParens = getOperatorParens(KernelObj);
31553156
assert(OpParens && "invalid kernel object");
31563157

0 commit comments

Comments
 (0)