diff options
| author | Peter Klausler <pklausler@nvidia.com> | 2022-10-31 14:17:02 -0700 |
|---|---|---|
| committer | Peter Klausler <pklausler@nvidia.com> | 2022-10-31 14:18:32 -0700 |
| commit | ad8f22c3fd4e92c69f6d0860ac5b3969f6a7aa9f (patch) | |
| tree | cefd731dc766ff85752834bb04a295da60f9d9ac /flang/lib/Semantics/pointer-assignment.cpp | |
| parent | 179978d7b8ec00291401d2ec49fc0a55e7f7bfb3 (diff) | |
| download | llvm-ad8f22c3fd4e92c69f6d0860ac5b3969f6a7aa9f.zip llvm-ad8f22c3fd4e92c69f6d0860ac5b3969f6a7aa9f.tar.gz llvm-ad8f22c3fd4e92c69f6d0860ac5b3969f6a7aa9f.tar.bz2 | |
[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/lib/Semantics/pointer-assignment.cpp')
| -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 7661a36..9ac18b0 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, |