<html><head></head><body>Yes, I'm happy. <br>
<br>
LGTM<br><br><div class="gmail_quote">On 20 June 2014 10:20:04 EEST, James Molloy <james.molloy@arm.com> wrote:<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<pre class="k9mail">Thanks Renato!<br /><br />Alp, are you happy too?<br /><br />Cheers,<br /><br />James<br /><br /><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;"> -----Original Message-----<br /> From: Renato Golin [mailto:renato.golin@linaro.org]<br /> Sent: 19 June 2014 17:08<br /> To: James Molloy<br /> Cc: Tim Northover; Clang Commits<br /> Subject: Re: [PATCH] Warn on use of vector initializers in ARM BE mode<br /> <br /> LGTM<br /> <br /> On 19 June 2014 16:27, James Molloy <james.molloy@arm.com> wrote:<br /><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #ad7fa8; padding-left: 1ex;"> Hi chaps,<br /><br /> So I've edited the patch again, taking into account Alp's comments.<br /><br /> The behaviour of the patch is that now, it will warn when encountering<br /></blockquote> an initializer on a NEON type (not just any ext_vector) in big endian<br /> mod!
e.<br
/><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #ad7fa8; padding-left: 1ex;"><br /> I think the impact of this is now sufficiently small that there should<br /></blockquote> be no complaints, and it could save users a lot of anguish.<br /><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #ad7fa8; padding-left: 1ex;"><br /> Cheers,<br /><br /> James<br /><br /><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #8ae234; padding-left: 1ex;"> -----Original Message-----<br /> From: Tim Northover [mailto:t.p.northover@gmail.com]<br /> Sent: 19 June 2014 14:53<br /> To: Renato Golin<br /> Cc: James Molloy; Clang Commits<br /> Subject: Re: [PATCH] Warn on use of vector initializers in ARM BE mode<br /><br /><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #fcaf3e; padding-left: 1ex;"> Tim, do you have any use case for the lac!
k of the
header, or is it<br /> just theoretical?<br /></blockquote><br /> Nah, it's purely theoretical. I don't really object to the patch<br /> either, so go ahead if people think it's a good idea.<br /><br /> Cheers.<br /><br /> Tim.<br /></blockquote></blockquote></blockquote><br /><br /><br /><br /></pre></blockquote></div></body></html>