<br><br><div class="gmail_quote">On Fri Jan 16 2015 at 12:08:21 PM Quentin Colombet <<a href="mailto:qcolombet@apple.com">qcolombet@apple.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Fiona,<br>
<br>
The approach looks overall good to me, but I have a few remarks and questions.<br>
1. Run your patch through clang-format please. The patch does not follow the LLVM formatting guidelines.<br>
2. What is the impact of this on arm64 and armv7s generated code? Although the approach makes sense to me, I want to be sure we do not degrade other targets. Note that I do not expect you to run tests if you cannot :).<br>
3. What are the runtime performance impact on x86_64, with and without -mavx2?<br>
4. Add a run line for avx  too:<br>
+; RUN: llc < %s -march=x86-64 -mattr=+avx2 | FileCheck %s<br>
5. Should we use a triple instead of march?<br>
<br></blockquote><div><br></div><div>I'll reply to this part: Probably not. The code generation likely shouldn't be to be target specific past x86_64. If it is then we've probably got something else to look at ;)</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">
Thanks,<br>
-Quentin<br>
<br>
<br>
On Jan 16, 2015, at 11:24 AM, Fiona Glaser <<a href="mailto:fglaser@apple.com" target="_blank">fglaser@apple.com</a>> wrote:<br>
<br>
> Talked about this one offline and decided to implement Mehdi’s strategy.<br>
><br>
> Fiona<br>
><br>
> <patch.diff>__________________<u></u>_____________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">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/<u></u>mailman/listinfo/llvm-commits</a><br>
<br>
<br>
______________________________<u></u>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">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/<u></u>mailman/listinfo/llvm-commits</a><br>
</blockquote></div>