<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">Hi,</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">
I'd like to say it is not a new feature at all. How should we define a new feature? What I said is "Complete AArch64 NEON feature" rather than "new AArch64 NEON feature".</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">
<br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">If we look at the list for LLVM patches, most of them are bug fixes addressing compiler crash. Those fixes makes AArch64 NEON more robustness.</div>
<div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">If we don't apply those patches, AArch64 NEON would often crash. It is bug fix or new feature? How do you judge?</div>
<div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">You may have a try by simply applying those two monolithic patches I provided. You won't see any failure on LLVM side.</div>
<div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">I think the purpose of this new 3.4.1 release is to fix all of the known bugs and bring value to end-user. I would be sad if we don't bring AArch64 NEON fix into this release, and end-user would be also sad, I believe.</div>
<div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">Thanks,</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">
-Jiangning</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small"><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-04-01 17:30 GMT+08:00 Sylvestre Ledru <span dir="ltr"><<a href="mailto:sylvestre@debian.org" target="_blank">sylvestre@debian.org</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On 01/04/2014 11:26, Renato Golin wrote:<br>
> Hi Jiangning,<br>
><br>
> Thank you very much for doing all the hard work of integrating the<br>
> patches and testing it beforehand, it really helps the back-porting!<br>
><br>
><br>
> On 1 April 2014 09:25, Jiangning Liu <<a href="mailto:liujiangning1@gmail.com">liujiangning1@gmail.com</a>> wrote:<br>
>> I know the patch list is little bit longer, we have the following reasons,<br>
>> 1) Last year, when branch 3.4 was created, actually we didn't really have<br>
>> time to complete all AArch64 neon work. So branch 3.4 is actually at the<br>
>> middle stage of aarch64 neon implementation. Now the patches I'm requesting<br>
>> intends to give a complete AArch64 NEON feature.<br>
> I'd be less comfortable with this change than the others, since it<br>
> introduces new functionality (and may introduce incompatibility with<br>
> previous behaviour), but I'll let Tom decide.<br>
</div>With my packager hat, I agree with you. I don't think we should accept<br>
any new features in the stable releases.<br>
Only important bugs fixes should be part of the release.<br>
<span class="HOEnZb"><font color="#888888"><br>
Sylvestre<br>
<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><font face="courier new, monospace">Thanks,</font><div><font face="courier new, monospace">-Jiangning</font></div></div>
</div>