[llvm] r345169 - [Hexagon] Flip hexagon-autohvx to be true by default

[Krzysztof Parzyszek via llvm-commits]

Hi Alina,
Feel free to revert, but please revert both patches.
I can do it, but it'll have to wait until Monday.

-Krzysztof

On 10/26/2018 6:03 PM, Alina Sbirlea wrote:
> FYI, this breaks some of our Halide tests. We're seeing timeouts (tests 
> hanging or taking a very large amount of time to compile).
> We're looking to obtain a reproducer, but reverting would be good in 
> order to unblock folks.
> 
> On Wed, Oct 24, 2018 at 10:57 AM Krzysztof Parzyszek via llvm-commits 
> <llvm-commits at lists.llvm.org <mailto:llvm-commits at lists.llvm.org>> wrote:
> 
>     Author: kparzysz
>     Date: Wed Oct 24 10:55:13 2018
>     New Revision: 345169
> 
>     URL: http://llvm.org/viewvc/llvm-project?rev=345169&view=rev
>     Log:
>     [Hexagon] Flip hexagon-autohvx to be true by default
> 
>     This will allow other generators of LLVM IR to use the auto-vectorizer
>     without having to change that flag.
> 
>     Note: on its own, this patch will enable auto-vectorization on Hexagon
>     in all cases, regardless of the -fvectorize flag. There is a companion
>     clang patch that together with this one forms an NFC for clang users.
> 
>     Modified:
>          llvm/trunk/lib/Target/Hexagon/HexagonTargetTransformInfo.cpp
> 
>     Modified: llvm/trunk/lib/Target/Hexagon/HexagonTargetTransformInfo.cpp
>     URL:
>     http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Hexagon/HexagonTargetTransformInfo.cpp?rev=345169&r1=345168&r2=345169&view=diff
>     ==============================================================================
>     --- llvm/trunk/lib/Target/Hexagon/HexagonTargetTransformInfo.cpp
>     (original)
>     +++ llvm/trunk/lib/Target/Hexagon/HexagonTargetTransformInfo.cpp Wed
>     Oct 24 10:55:13 2018
>     @@ -28,7 +28,7 @@ using namespace llvm;
> 
>       #define DEBUG_TYPE "hexagontti"
> 
>     -static cl::opt<bool> HexagonAutoHVX("hexagon-autohvx", cl::init(false),
>     +static cl::opt<bool> HexagonAutoHVX("hexagon-autohvx", cl::init(true),
>         cl::Hidden, cl::desc("Enable loop vectorizer for HVX"));
> 
>       static cl::opt<bool> EmitLookupTables("hexagon-emit-lookup-tables",
> 
> 
>     _______________________________________________
>     llvm-commits mailing list
>     llvm-commits at lists.llvm.org <mailto:llvm-commits at lists.llvm.org>
>     http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
> 
> 
> <http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=emailclient> 
> 	Virus-free. www.avg.com 
> <http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=emailclient> 
> 
> 
> <#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>