Skip to content

Commit 26bbc04

Browse files
Apply review comments - add missing fullstops in comments
Signed-off-by: Elizabeth Andrews <elizabeth.andrews@intel.com>
1 parent 26b2e52 commit 26bbc04

File tree

4 files changed

+6
-6
lines changed

4 files changed

+6
-6
lines changed

clang/lib/Sema/SemaDeclAttr.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3115,13 +3115,13 @@ Sema::mergeSYCLIntelLoopFuseAttr(Decl *D, const AttributeCommonInfo &CI,
31153115

31163116
assert(ArgVal && ExistingArgVal &&
31173117
"Argument should be an integer constant expression");
3118-
// Compare attribute argument value and warn if there is a mismatch
3118+
// Compare attribute argument value and warn if there is a mismatch.
31193119
if (ArgVal->getExtValue() != ExistingArgVal->getExtValue())
31203120
Diag(ExistingAttr->getLoc(), diag::warn_duplicate_attribute)
31213121
<< ExistingAttr;
31223122
}
31233123

3124-
// If there is no mismatch, silently ignore duplicate attribute
3124+
// If there is no mismatch, silently ignore duplicate attribute.
31253125
return nullptr;
31263126
}
31273127
return ::new (Context) SYCLIntelLoopFuseAttr(Context, CI, E);
@@ -3166,7 +3166,7 @@ void Sema::addSYCLIntelLoopFuseAttr(Decl *D, const AttributeCommonInfo &CI,
31663166

31673167
// Handles [[intel::loop_fuse]] and [[intel::loop_fuse_independent]].
31683168
static void handleLoopFuseAttr(Sema &S, Decl *D, const ParsedAttr &Attr) {
3169-
// Default argument value is set to 1
3169+
// Default argument value is set to 1.
31703170
Expr *E = Attr.isArgExpr(0)
31713171
? Attr.getArgAsExpr(0)
31723172
: IntegerLiteral::Create(S.Context, llvm::APInt(32, 1),

clang/lib/Sema/SemaSYCL.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -571,7 +571,7 @@ class MarkDeviceFunction : public RecursiveASTVisitor<MarkDeviceFunction> {
571571
FD->addAttr(SYCLSimdAttr::CreateImplicit(SemaRef.getASTContext()));
572572

573573
// Attribute "loop_fuse" can be applied explicitly on kernel function.
574-
// Attribute should not be propagated from device functions to kernel
574+
// Attribute should not be propagated from device functions to kernel.
575575
if (auto *A = FD->getAttr<SYCLIntelLoopFuseAttr>()) {
576576
if (ParentFD == SYCLKernel) {
577577
Attrs.insert(A);

clang/test/CodeGenSYCL/loop_fuse_device.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ class KernelFunctor5 {
1616
void bar() {
1717

1818
q.submit([&](handler &h) {
19-
// Test template argument
19+
// Test template argument.
2020
KernelFunctor5<5> f5;
2121
h.single_task<class kernel_name_1>(f5);
2222

clang/test/CodeGenSYCL/loop_fuse_ind_device.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ class KernelFunctor5 {
1616
void bar() {
1717

1818
q.submit([&](handler &h) {
19-
// Test template argument
19+
// Test template argument.
2020
KernelFunctor5<5> f5;
2121
h.single_task<class kernel_name_1>(f5);
2222

0 commit comments

Comments
 (0)