From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 74886 invoked by alias); 12 Nov 2015 20:04:04 -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 74870 invoked by uid 89); 12 Nov 2015 20:04:03 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.5 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 X-HELO: mail-qk0-f177.google.com Received: from mail-qk0-f177.google.com (HELO mail-qk0-f177.google.com) (209.85.220.177) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-GCM-SHA256 encrypted) ESMTPS; Thu, 12 Nov 2015 20:04:02 +0000 Received: by qkfo3 with SMTP id o3so27268055qkf.1 for ; Thu, 12 Nov 2015 12:04:00 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:date :message-id:subject:from:to:cc:content-type; bh=sPBeNjJsDg77vkTkWCwdWUSvhL3uxxTBnLccdB5Od3E=; b=VEv8nxyNwPHjBGxp3XD5npXj8rNnIUw3PqLpnDLW2jL0BXDWaL/aNmXFKccgq9eQQz hvWhUizU3D1ci2HI0ptBfGDe9QsCWwbrIW+cc2y+g9GKuLLLV9t1HbLEloOSYgZ5yZ6J IOVCXleuemOoY4PFORQVrpK9QIlEfNUNS4boVURsQrrxWXOPVFWSGQxXkQTc0mGex5r8 dcFRfydu7igGxxZyiZnyaeSHW4fwTco96pthdRKMHEanXu0A0btYIx8Qf6BkwZWEt/fs oCsIhHMRfAGrS59xDYkZeqBT7fsZEzGzlZAVLqO8+umT6REEM1pE2e46YcJn9rBsTZ+J rCjA== X-Gm-Message-State: ALoCoQmIsi0XNfCCQFiCwUv6PydM0EhdSAjaJJdDHl8KGoovdCHuH2C995gubtuN5KsP2pnu5fLk MIME-Version: 1.0 X-Received: by 10.55.33.40 with SMTP id h40mr17795211qkh.77.1447358640291; Thu, 12 Nov 2015 12:04:00 -0800 (PST) Received: by 10.140.109.200 with HTTP; Thu, 12 Nov 2015 12:04:00 -0800 (PST) In-Reply-To: References: Date: Thu, 12 Nov 2015 20:04:00 -0000 Message-ID: Subject: Re: [PATCH] More compile-time saving in BB vectorization From: Christophe Lyon To: Andreas Schwab Cc: Richard Biener , "gcc-patches@gcc.gnu.org" Content-Type: text/plain; charset=UTF-8 X-IsSubscribed: yes X-SW-Source: 2015-11/txt/msg01589.txt.bz2 On 12 November 2015 at 16:49, Andreas Schwab wrote: > 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 > Same problem on armeb. > 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."