Skip to content

Commit c16dc63

Browse files
committed
[OMPIRBuilder] Fix gcc -Wparentheses warning [NFC]
Without this gcc warned like /repo/llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp:7559:68: warning: suggest parentheses around '&&' within '||' [-Wparentheses] 7559 | NumStaleCIArgs == (OffloadingArraysToPrivatize.size() + 2) && | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~ 7560 | "Wrong number of arguments for StaleCI when shareds are present"); | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 parent 7ea7ccd commit c16dc63

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7558,10 +7558,9 @@ OpenMPIRBuilder::InsertPointOrErrorTy OpenMPIRBuilder::emitTargetTask(
75587558
// StaleCI is exactly OffloadingArraysToPrivatize.size() + 2
75597559
const unsigned int NumStaleCIArgs = StaleCI->arg_size();
75607560
bool HasShareds = NumStaleCIArgs > OffloadingArraysToPrivatize.size() + 1;
7561-
assert(
7562-
!HasShareds ||
7563-
NumStaleCIArgs == (OffloadingArraysToPrivatize.size() + 2) &&
7564-
"Wrong number of arguments for StaleCI when shareds are present");
7561+
assert((!HasShareds ||
7562+
NumStaleCIArgs == (OffloadingArraysToPrivatize.size() + 2)) &&
7563+
"Wrong number of arguments for StaleCI when shareds are present");
75657564
int SharedArgOperandNo =
75667565
HasShareds ? OffloadingArraysToPrivatize.size() + 1 : 0;
75677566

0 commit comments

Comments
 (0)