diff options
author | Benjamin Kramer <benny.kra@googlemail.com> | 2016-02-18 13:23:17 +0000 |
---|---|---|
committer | Benjamin Kramer <benny.kra@googlemail.com> | 2016-02-18 13:23:17 +0000 |
commit | e593094a1524789330f70a12f38405c27c75ceec (patch) | |
tree | b8907b0522bb6580d79d85e457984b900fb545f1 | |
parent | 3592a6ec6bdc7104c6d0e6140293eb02637e3f75 (diff) | |
download | bcm5719-llvm-e593094a1524789330f70a12f38405c27c75ceec.tar.gz bcm5719-llvm-e593094a1524789330f70a12f38405c27c75ceec.zip |
Add parentheses around arithmetic in operand of '|'.
This avoids a operator precedence warning for mixing + and | in an
expression. I checked that this matches the definition in the Split
DWARF proposal.
Patch by Cong Liu!
Differential Revision: http://reviews.llvm.org/D17375
llvm-svn: 261207
-rw-r--r-- | llvm/tools/llvm-dwp/llvm-dwp.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/tools/llvm-dwp/llvm-dwp.cpp b/llvm/tools/llvm-dwp/llvm-dwp.cpp index 02fcc151c05..9bc32760591 100644 --- a/llvm/tools/llvm-dwp/llvm-dwp.cpp +++ b/llvm/tools/llvm-dwp/llvm-dwp.cpp @@ -238,7 +238,7 @@ static void writeIndex(MCStreamer &Out, MCSection *Section, while (Buckets[H]) { assert(S != IndexEntries[Buckets[H] - 1].Signature && "Duplicate type unit"); - H = (H + ((S >> 32) & Mask) | 1) % Buckets.size(); + H = (H + (((S >> 32) & Mask) | 1)) % Buckets.size(); } Buckets[H] = i + 1; } |