<div dir="ltr"><div>Patch was reformatted and committed @r212520<br></div><div><br></div><div>Thanks Jim !<br></div><div><br></div><div>Cheers,</div><div>--</div><div>Arnaud</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Tue, Jul 8, 2014 at 1:17 AM, Jim Grosbach <span dir="ltr"><<a href="mailto:grosbach@apple.com" target="_blank">grosbach@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Seems reasonable to me. Some minor formatting cleanups. Easiest to just run clang-format over the lines of code you’re changing for that.<br>
<br>
-Jim<br>
<div><div class="h5"><br>
<br>
> On Jul 7, 2014, at 2:42 PM, Arnaud Allard de Grandmaison <<a href="mailto:arnaud.adegm@gmail.com">arnaud.adegm@gmail.com</a>> wrote:<br>
><br>
> Logical 32 bit operations should truncate the immediate to the register witdth, i.e. 32 bits.<br>
><br>
> This allows MC assembler to handle instructions like 'and w2, w3, #-3'.<br>
><br>
> The truncation is not applied blindly : the 32 most significant bits are checked to be all ones or zeros so that we catch programming errors.<br>
><br>
> Comments ? OK to apply ?<br>
><br>
> Cheers,<br>
> --<br>
> Arnaud de Grandmaison<br>
><br>
</div></div>> <0001-AArch64-Truncate-the-immediate-in-logical-operation-.patch>_______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br>
</blockquote></div><br></div>