From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 104208 invoked by alias); 12 Nov 2015 15:49:23 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 104131 invoked by uid 89); 12 Nov 2015 15:49:22 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-3.0 required=5.0 tests=AWL,BAYES_00,RP_MATCHES_RCVD,SPF_PASS autolearn=ham version=3.3.2 X-HELO: mx2.suse.de Received: from mx2.suse.de (HELO mx2.suse.de) (195.135.220.15) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (CAMELLIA256-SHA encrypted) ESMTPS; Thu, 12 Nov 2015 15:49:21 +0000 Received: from relay2.suse.de (charybdis-ext.suse.de [195.135.220.254]) by mx2.suse.de (Postfix) with ESMTP id 85080ABE5 for ; Thu, 12 Nov 2015 15:48:55 +0000 (UTC) From: Andreas Schwab To: Richard Biener Cc: gcc-patches@gcc.gnu.org Subject: Re: [PATCH] More compile-time saving in BB vectorization References: X-Yow: Talking Pinhead Blues: Oh, I LOST my ``HELLO KITTY'' DOLL and I get BAD reception on channel TWENTY-SIX!! Th'HOSTESS FACTORY is closin' down and I just heard ZASU PITTS has been DEAD for YEARS.. (sniff) My PLATFORM SHOE collection was CHEWED up by th'dog, ALEXANDER HAIG won't let me take a SHOWER 'til Easter.. (snurf) So I went to the kitchen, but WALNUT PANELING whup me upside mah HAID!! (on no, no, no.. Heh, heh) Date: Thu, 12 Nov 2015 15:49:00 -0000 In-Reply-To: (Richard Biener's message of "Wed, 11 Nov 2015 15:29:19 +0100 (CET)") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-SW-Source: 2015-11/txt/msg01552.txt.bz2 Richard Biener writes: > * tree-vectorizer.h (vect_slp_analyze_and_verify_instance_alignment): > Declare. > (vect_analyze_data_refs_alignment): Make loop vect specific. > (vect_verify_datarefs_alignment): Likewise. > * tree-vect-data-refs.c (vect_slp_analyze_data_ref_dependences): > Add missing continue. > (vect_compute_data_ref_alignment): Export. > (vect_compute_data_refs_alignment): Merge into... > (vect_analyze_data_refs_alignment): ... this. > (verify_data_ref_alignment): Split out from ... > (vect_verify_datarefs_alignment): ... here. > (vect_slp_analyze_and_verify_node_alignment): New function. > (vect_slp_analyze_and_verify_instance_alignment): Likewise. > * tree-vect-slp.c (vect_supported_load_permutation_p): Remove > misplaced checks on alignment. > (vect_slp_analyze_bb_1): Add fatal output parameter. Do > alignment analysis after SLP discovery and do it per instance. > (vect_slp_bb): When vect_slp_analyze_bb_1 fatally failed do not > bother to re-try using different vector sizes. This breaks libgfortran on ia64: ../../../libgfortran/generated/matmul_c4.c: In function 'matmul_c4': ../../../libgfortran/generated/matmul_c4.c:79:1: internal compiler error: in vectorizable_store, at tree-vect-stmts.c:5651 matmul_c4 (gfc_array_c4 * const restrict retarray, ^ 0x40000000010ff01f vectorizable_store ../../gcc/tree-vect-stmts.c:5651 0x4000000001115b5f vect_transform_stmt(gimple*, gimple_stmt_iterator*, bool*, _slp_tree*, _slp_instance*) ../../gcc/tree-vect-stmts.c:8003 0x400000000114df1f vect_schedule_slp_instance ../../gcc/tree-vect-slp.c:3484 0x4000000001154d6f vect_schedule_slp(vec_info*) ../../gcc/tree-vect-slp.c:3549 0x40000000011562bf vect_slp_bb(basic_block_def*) ../../gcc/tree-vect-slp.c:2543 0x4000000001159f2f execute ../../gcc/tree-vectorizer.c:734 Andreas. -- Andreas Schwab, SUSE Labs, schwab@suse.de GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE 1748 E4D4 88E3 0EEA B9D7 "And now for something completely different."