Skip to content

Commit fe13de3

Browse files
committed
More compact, yet still awkward loop counter update
1 parent 2eac67e commit fe13de3

File tree

1 file changed

+6
-12
lines changed

1 file changed

+6
-12
lines changed

src/basic/DAXPY-Seq.cpp

Lines changed: 6 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -37,10 +37,8 @@ void DAXPY::runSeqVariant(VariantID vid, size_t RAJAPERF_UNUSED_ARG(tune_idx))
3737
case Base_Seq : {
3838

3939
startTimer();
40-
// Standalone assignment expression for loop counter to quiet
41-
// C++20 compiler warning
42-
for (RepIndex_type irep = 0; irep < run_reps; /* no update here */) {
43-
irep = irep + 1;
40+
// Awkward expression for loop counter quiets C++20 compiler warning
41+
for (RepIndex_type irep = 0; irep < run_reps; ((irep = irep + 1), 0)) {
4442

4543
for (Index_type i = ibegin; i < iend; ++i ) {
4644
DAXPY_BODY;
@@ -56,10 +54,8 @@ void DAXPY::runSeqVariant(VariantID vid, size_t RAJAPERF_UNUSED_ARG(tune_idx))
5654
case Lambda_Seq : {
5755

5856
startTimer();
59-
// Standalone assignment expression for loop counter to quiet
60-
// C++20 compiler warning
61-
for (RepIndex_type irep = 0; irep < run_reps; /* no update here */) {
62-
irep = irep + 1;
57+
// Awkward expression for loop counter quiets C++20 compiler warning
58+
for (RepIndex_type irep = 0; irep < run_reps; ((irep = irep + 1), 0)) {
6359

6460
for (Index_type i = ibegin; i < iend; ++i ) {
6561
daxpy_lam(i);
@@ -76,10 +72,8 @@ void DAXPY::runSeqVariant(VariantID vid, size_t RAJAPERF_UNUSED_ARG(tune_idx))
7672
auto res{getHostResource()};
7773

7874
startTimer();
79-
// Standalone assignment expression for loop counter to quiet
80-
// C++20 compiler warning
81-
for (RepIndex_type irep = 0; irep < run_reps; /* no update here */) {
82-
irep = irep + 1;
75+
// Awkward expression for loop counter quiets C++20 compiler warning
76+
for (RepIndex_type irep = 0; irep < run_reps; ((irep = irep + 1), 0)) {
8377

8478
RAJA::forall<RAJA::seq_exec>( res,
8579
RAJA::RangeSegment(ibegin, iend), daxpy_lam);

0 commit comments

Comments
 (0)