summaryrefslogtreecommitdiffstats
path: root/clang/lib/Driver/Phases.cpp
diff options
context:
space:
mode:
authorNico Weber <thakis@chromium.org>2020-02-03 14:16:52 -0500
committerHans Wennborg <hans@chromium.org>2020-02-19 13:27:51 +0100
commit8de07c31c1aafa848f515d721e6cf065a0701e81 (patch)
tree9576685367d86c8610c902f0a842d76c1b53c824 /clang/lib/Driver/Phases.cpp
parent3dee8be1c3fe621b9c1926658f6c0df72ce804be (diff)
downloadbcm5719-llvm-8de07c31c1aafa848f515d721e6cf065a0701e81.tar.gz
bcm5719-llvm-8de07c31c1aafa848f515d721e6cf065a0701e81.zip
Fix a -Wbitwise-conditional-parentheses warning in _LIBUNWIND_ARM_EHABI libunwind builds
``` src/UnwindCursor.hpp:1344:51: error: operator '?:' has lower precedence than '|'; '|' will be evaluated first [-Werror,-Wbitwise-conditional-parentheses] _info.flags = isSingleWordEHT ? 1 : 0 | scope32 ? 0x2 : 0; // Use enum? ~~~~~~~~~~~ ^ src/UnwindCursor.hpp:1344:51: note: place parentheses around the '|' expression to silence this warning _info.flags = isSingleWordEHT ? 1 : 0 | scope32 ? 0x2 : 0; // Use enum? ^ ( ) src/UnwindCursor.hpp:1344:51: note: place parentheses around the '?:' expression to evaluate it first _info.flags = isSingleWordEHT ? 1 : 0 | scope32 ? 0x2 : 0; // Use enum? ^ ( ) ``` But `0 |` is a no-op for either of those two interpretations, so I think what was meant here was ``` _info.flags = (isSingleWordEHT ? 1 : 0) | (scope32 ? 0x2 : 0); // Use enum? ``` Previously, if `isSingleWordEHT` was set, bit 2 would never be set. Now it is. From what I can tell, the only thing that checks these bitmask is ProcessDescriptors in Unwind-EHABI.cpp, and that only cares about bit 1, so in practice this shouldn't have much of an effect. Differential Revision: https://reviews.llvm.org/D73890 (cherry picked from commit 221c5af4e4f4a504a4d1f352dd7b76d305e56a62)
Diffstat (limited to 'clang/lib/Driver/Phases.cpp')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud