From e593094a1524789330f70a12f38405c27c75ceec Mon Sep 17 00:00:00 2001 From: Benjamin Kramer Date: Thu, 18 Feb 2016 13:23:17 +0000 Subject: [PATCH] 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 --- llvm/tools/llvm-dwp/llvm-dwp.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/llvm/tools/llvm-dwp/llvm-dwp.cpp b/llvm/tools/llvm-dwp/llvm-dwp.cpp index 02fcc151c050..9bc327605910 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; }