<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Wed, May 31, 2017 at 6:35 AM Renato Golin <<a href="mailto:renato.golin@linaro.org">renato.golin@linaro.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 31 May 2017 at 13:57, Kristof Beyls <<a href="mailto:Kristof.Beyls@arm.com" target="_blank">Kristof.Beyls@arm.com</a>> wrote:<br>
> Taking into account the above measurements, my conclusions are:<br>
> 1. We should make -mcpu=generic the default cpu, not Cortex-A8 or Cortex-A53<br>
> for march=armv7a and march=armv8a.<br>
<br>
Using -mcpu=native makes more sense to me, if at all possible to<br>
detect, falling back to generic, which doesn't hurt.<br>
<br></blockquote><div><br></div><div>Ultimately either solution is fine with me. If Kristof wanted to switch it to generic while getting the autodetection stuff up that would also be ok.</div><div><br></div><div>-eric</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
> 2. We probably want to let the compiler schedule instructions using the<br>
> Cortex-A8 pipeline model for -mcpu=generic, since it gives a bit of speedup<br>
> on all cores tested.<br>
<br>
Same here, I'd use the schedule of the detected CPU, if any, or fall<br>
back to A8 (which seems fine).<br>
<br>
But yeah, it's time we get rid of the A8/A53 defaults.<br>
<br>
While we're at it, we may think about ARMv7's NEON default. Generating<br>
only VFP is slower on boards with NEON, but generating NEON crashes<br>
with SIGILL on borads that don't have it.<br>
<br>
I'd be happy if Clang could detect CPU/FPU and set the flags<br>
accordingly, or fall back to "generic"/A8-schedule/VFP defaults.<br>
<br>
cheers,<br>
--renato<br>
</blockquote></div></div>