<div dir="ltr"><div dir="ltr"><div dir="ltr">Hi Krzysztof,<div><br></div><div>Reverted in <a href="https://reviews.llvm.org/rL345446">r345446</a> and <a href="https://reviews.llvm.org/rL345447">r345447</a>.</div><div><br></div><div>Thank you,</div><div>Alina</div><div><br></div></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Oct 26, 2018 at 5:35 PM Krzysztof Parzyszek <<a href="mailto:kparzysz@codeaurora.org">kparzysz@codeaurora.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Alina,<br>
Feel free to revert, but please revert both patches.<br>
I can do it, but it'll have to wait until Monday.<br>
<br>
-Krzysztof<br>
<br>
On 10/26/2018 6:03 PM, Alina Sbirlea wrote:<br>
> FYI, this breaks some of our Halide tests. We're seeing timeouts (tests <br>
> hanging or taking a very large amount of time to compile).<br>
> We're looking to obtain a reproducer, but reverting would be good in <br>
> order to unblock folks.<br>
> <br>
> On Wed, Oct 24, 2018 at 10:57 AM Krzysztof Parzyszek via llvm-commits <br>
> <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a> <mailto:<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>>> wrote:<br>
> <br>
>     Author: kparzysz<br>
>     Date: Wed Oct 24 10:55:13 2018<br>
>     New Revision: 345169<br>
> <br>
>     URL: <a href="http://llvm.org/viewvc/llvm-project?rev=345169&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=345169&view=rev</a><br>
>     Log:<br>
>     [Hexagon] Flip hexagon-autohvx to be true by default<br>
> <br>
>     This will allow other generators of LLVM IR to use the auto-vectorizer<br>
>     without having to change that flag.<br>
> <br>
>     Note: on its own, this patch will enable auto-vectorization on Hexagon<br>
>     in all cases, regardless of the -fvectorize flag. There is a companion<br>
>     clang patch that together with this one forms an NFC for clang users.<br>
> <br>
>     Modified:<br>
>          llvm/trunk/lib/Target/Hexagon/HexagonTargetTransformInfo.cpp<br>
> <br>
>     Modified: llvm/trunk/lib/Target/Hexagon/HexagonTargetTransformInfo.cpp<br>
>     URL:<br>
>     <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Hexagon/HexagonTargetTransformInfo.cpp?rev=345169&r1=345168&r2=345169&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Hexagon/HexagonTargetTransformInfo.cpp?rev=345169&r1=345168&r2=345169&view=diff</a><br>
>     ==============================================================================<br>
>     --- llvm/trunk/lib/Target/Hexagon/HexagonTargetTransformInfo.cpp<br>
>     (original)<br>
>     +++ llvm/trunk/lib/Target/Hexagon/HexagonTargetTransformInfo.cpp Wed<br>
>     Oct 24 10:55:13 2018<br>
>     @@ -28,7 +28,7 @@ using namespace llvm;<br>
> <br>
>       #define DEBUG_TYPE "hexagontti"<br>
> <br>
>     -static cl::opt<bool> HexagonAutoHVX("hexagon-autohvx", cl::init(false),<br>
>     +static cl::opt<bool> HexagonAutoHVX("hexagon-autohvx", cl::init(true),<br>
>         cl::Hidden, cl::desc("Enable loop vectorizer for HVX"));<br>
> <br>
>       static cl::opt<bool> EmitLookupTables("hexagon-emit-lookup-tables",<br>
> <br>
> <br>
>     _______________________________________________<br>
>     llvm-commits mailing list<br>
>     <a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a> <mailto:<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>><br>
>     <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
> <br>
> <br>
> <<a href="http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=emailclient" rel="noreferrer" target="_blank">http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=emailclient</a>> <br>
>       Virus-free. <a href="http://www.avg.com" rel="noreferrer" target="_blank">www.avg.com</a> <br>
> <<a href="http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=emailclient" rel="noreferrer" target="_blank">http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=emailclient</a>> <br>
> <br>
> <br>
> <#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2><br>
<br>
</blockquote></div>