[PATCH] D17375: Add parentheses around arithmetic in operand of '|' in llvm-dwp.cpp.
Cong Liu via cfe-commits
cfe-commits at lists.llvm.org
Thu Feb 18 05:20:15 PST 2016
congliu updated this revision to Diff 48301.
congliu added a comment.
- Address review comment.
http://reviews.llvm.org/D17375
Files:
tools/llvm-dwp/llvm-dwp.cpp
Index: tools/llvm-dwp/llvm-dwp.cpp
===================================================================
--- tools/llvm-dwp/llvm-dwp.cpp
+++ tools/llvm-dwp/llvm-dwp.cpp
@@ -238,7 +238,7 @@
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;
}
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D17375.48301.patch
Type: text/x-patch
Size: 467 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160218/dbf69d6a/attachment.bin>
More information about the cfe-commits
mailing list