[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