<div dir="ltr">Thanks all! Which compiler flagged this? Wonder if/why Clang didn't flag it for me?</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Feb 18, 2016 at 5:27 AM, Phabricator via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This revision was automatically updated to reflect the committed changes.<br>
Closed by commit rL261207: Add parentheses around arithmetic in operand of '|'. (authored by d0k).<br>
<br>
Changed prior to commit:<br>
  <a href="http://reviews.llvm.org/D17375?vs=48301&id=48302#toc" rel="noreferrer" target="_blank">http://reviews.llvm.org/D17375?vs=48301&id=48302#toc</a><br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D17375" rel="noreferrer" target="_blank">http://reviews.llvm.org/D17375</a><br>
<br>
Files:<br>
  llvm/trunk/tools/llvm-dwp/llvm-dwp.cpp<br>
<br>
Index: llvm/trunk/tools/llvm-dwp/llvm-dwp.cpp<br>
===================================================================<br>
--- llvm/trunk/tools/llvm-dwp/llvm-dwp.cpp<br>
+++ llvm/trunk/tools/llvm-dwp/llvm-dwp.cpp<br>
<span class="">@@ -238,7 +238,7 @@<br>
     while (Buckets[H]) {<br>
       assert(S != IndexEntries[Buckets[H] - 1].Signature &&<br>
              "Duplicate type unit");<br>
-      H = (H + ((S >> 32) & Mask) | 1) % Buckets.size();<br>
</span>+      H = (H + (((S >> 32) & Mask) | 1)) % Buckets.size();<br>
<div class="HOEnZb"><div class="h5">     }<br>
     Buckets[H] = i + 1;<br>
   }<br>
<br>
<br>
</div></div><br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>