<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Jun 26, 2014 at 11:57 AM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">================<br>
Comment at: src/Unwind/Unwind-EHABI.cpp:213<br>
@@ -211,3 +212,3 @@<br>
 uint32_t RegisterRange(uint8_t start, uint8_t count_minus_one) {<br>
-  return (start << 16) | (count_minus_one + 1);<br>
+  return ((uint32_t)start << 16) | ((uint32_t)count_minus_one + 1);<br>
 }<br>
----------------<br>
</div><div class="">Nico Weber wrote:<br>
> Huh, your compiler warns when widening a unsigned byte to an unsigned int? Interesting.<br>
</div>Both subexpressions of the `|` used to be of type `signed int`; the warning will be for converting the result of the `|` to `unsigned`.<br></blockquote><div><br></div><div>Ah, right. Might be simpler to just add a u suffix to the two literals then.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<a href="http://reviews.llvm.org/D4315" target="_blank">http://reviews.llvm.org/D4315</a><br>
<br>
<br>
</blockquote></div><br></div></div>