<div dir="ltr">Hm, I checked "Accept revision" on phab, looks like that doesn't make it into the email. So here's an explicit "looks good to me" (the comments are just suggestions)</div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Thu, Jun 26, 2014 at 11:27 AM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Thanks!<br>
<br>
================<br>
Comment at: src/Unwind/Unwind-EHABI.cpp:110<br>
@@ -108,3 +109,3 @@<br>
     // Clear off flag from last bit.<br>
-    length &= ~1;<br>
-    offset &= ~1;<br>
+    length &= ~(uint32_t)0x1;<br>
+    offset &= ~(uint32_t)0x1;<br>
----------------<br>
No need for the 0x<br>
<br>
Also, since we know this is always 32bit code, you can just write ~1u (everywhere)<br>
<br>
================<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>
Huh, your compiler warns when widening a unsigned byte to an unsigned int? Interesting.<br>
<br>
<a href="http://reviews.llvm.org/D4315" target="_blank">http://reviews.llvm.org/D4315</a><br>
<br>
<br>
</blockquote></div><br></div>