diff options
author | Peter Klausler <pklausler@nvidia.com> | 2022-11-01 00:17:02 +0300 |
---|---|---|
committer | Peter Klausler <pklausler@nvidia.com> | 2022-11-01 00:18:32 +0300 |
commit | ad8f22c3fd4e92c69f6d0860ac5b3969f6a7aa9f (patch) | |
tree | cefd731dc766ff85752834bb04a295da60f9d9ac /flang | |
parent | 179978d7b8ec00291401d2ec49fc0a55e7f7bfb3 (diff) |
[flang] Rework line of code to dodge clang 16 warning
Recode a non-short-circuiting conjunction of two Boolean function calls
into separate statements to avoid a warning from clang 16.
Diffstat (limited to 'flang')
-rw-r--r-- | flang/lib/Semantics/pointer-assignment.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/flang/lib/Semantics/pointer-assignment.cpp b/flang/lib/Semantics/pointer-assignment.cpp index 7661a36faaef..9ac18b0d2c5b 100644 --- a/flang/lib/Semantics/pointer-assignment.cpp +++ b/flang/lib/Semantics/pointer-assignment.cpp @@ -429,7 +429,9 @@ bool CheckPointerAssignment(evaluate::FoldingContext &context, } PointerAssignmentChecker checker{context, scope, *pointer}; checker.set_isBoundsRemapping(isBoundsRemapping); - return checker.CheckLeftHandSide(lhs) & checker.Check(rhs); + bool lhsOk{checker.CheckLeftHandSide(lhs)}; + bool rhsOk{checker.Check(rhs)}; + return lhsOk && rhsOk; // don't short-circuit } bool CheckStructConstructorPointerComponent(evaluate::FoldingContext &context, |