<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jan 14, 2015 at 5:53 PM, Kevin Qin <span dir="ltr"><<a href="mailto:kevinqindev@gmail.com" target="_blank">kevinqindev@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi <span style="font-size:13.63636302948px">Michael and David,</span><div><span style="font-size:13.63636302948px"><br></span></div><div><span style="font-size:13.63636302948px">I also see some performance regressions on aarch64 target introduced by this commit. No further invesitgation on what's the root cause, but I guess the one Michael shared may cause regression on many targets, including aarch64.</span></div><div><span style="font-size:13.63636302948px"><br></span></div><div><span style="font-size:13.63636302948px">I argee with </span><span style="font-size:13.63636302948px">Michael's suggestion that</span><span style="font-size:13.63636302948px"> revert this commit firstly, and make more performance test over it.</span></div></div></blockquote></div><br>It would be good to understand whether the change David made early today would have addressed these issues or not.</div><div class="gmail_extra"><br></div><div class="gmail_extra">(Either way, it has made me suspect that we may want A - B < 0 to be the canonical form, and lower it cleverly in the backend.)</div></div>