From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 2941 invoked by alias); 9 Oct 2015 14:04:26 -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 2931 invoked by uid 89); 9 Oct 2015 14:04:25 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.4 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 X-HELO: mail-ob0-f178.google.com Received: from mail-ob0-f178.google.com (HELO mail-ob0-f178.google.com) (209.85.214.178) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-GCM-SHA256 encrypted) ESMTPS; Fri, 09 Oct 2015 14:04:24 +0000 Received: by obcgx8 with SMTP id gx8so63005312obc.3 for ; Fri, 09 Oct 2015 07:04:22 -0700 (PDT) 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=jQpJ+xD0bZBJWuwTN5I+KPz3fREh0eDbuUFbOV5QAzo=; b=PjtUsov6oZNFBtHO2DXJ4Wlr7LrtMUWwBYpWSYkJYkb0t4xN2HWFOna4oS57kwL7NX Iq+VPVLtUcqNpj3nC7QcvsFveyYhYpuo+g8EgBcLcJszF9ZYMfAKhAEeMLshK9FePKIg erB63txVjPz6lAZqxII7GRmqoDp1Ub2M/YxC6rRAJh36lxKCfeADuYbFAVcZzoh7uwT5 Ebdn2JlM/GxASQG24+5Gr7i1Slqu2b2em4xw1VgN6ivhZwOCgNIHz38G1676ZObIIWah tjTkuBS1tqIaF+tESMziRYRwQTWvH5sBm4CIywzxozFiIq2ZyDYYkVJuvO8SqXp9SmUM dr5g== X-Gm-Message-State: ALoCoQmR66D73VrAuoQ+Oi2F/7RCpRQE35gA0S5O8NJoK8y7HR5KPIIAJXu7N8ERWwcdHjcoAOuE MIME-Version: 1.0 X-Received: by 10.60.98.196 with SMTP id ek4mr8603629oeb.39.1444399462268; Fri, 09 Oct 2015 07:04:22 -0700 (PDT) Received: by 10.202.60.195 with HTTP; Fri, 9 Oct 2015 07:04:22 -0700 (PDT) In-Reply-To: <1444175989-24944-1-git-send-email-charles.baylis@linaro.org> References: <1444175989-24944-1-git-send-email-charles.baylis@linaro.org> Date: Fri, 09 Oct 2015 14:04:00 -0000 Message-ID: Subject: Re: [PATCH v2 0/3] [ARM] PR63870 vldN_lane/vstN_lane error messages From: Charles Baylis To: Kyrylo Tkachov , Ramana Radhakrishnan , Alan Lawrence Cc: GCC Patches Content-Type: text/plain; charset=UTF-8 X-IsSubscribed: yes X-SW-Source: 2015-10/txt/msg00978.txt.bz2 On 7 October 2015 at 00:59, wrote: > From: Charles Baylis > > This patch series fixes up the error messages for single lane vector > load/stores, similarly to AArch64. > > make check on arm-linux-gnueabihf/qemu completes with no new regressions. > > Changes since the last version: > . removed the duplicate arm_neon_lane_bounds function > . resolved conflicts with other NEON work > . whitespace clean up > A bit more info is required here. Original patch submission was at: https://gcc.gnu.org/ml/gcc-patches/2015-07/msg00142.html > Charles Baylis (3): > [ARM] PR63870 Add qualifiers for NEON builtins This adds the qualifier_struct_load_store_lane_index qualifier, ported from my AArch64 changes. This causes the lane bounds check to be performed and, on big-endian targets, reverses the lane number used in the RTL (in common with other NEON patterns). This isn't strictly required for correctness, since the vectorizer never generates these patterns, but the consistency seems like a good idea. > [ARM] PR63870 Mark lane indices of vldN/vstN with appropriate > qualifier This marks the builtins with qualifier_struct_load_store_lane_index. The patterns are also updated to un-reverse the lane order at assembly time. > [ARM] PR63870 Enable test cases for ARM Removes the xfails from the test cases.