public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
From: "cvs-commit at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug target/107714] MVE: Invalid addressing mode generated for VLD2
Date: Tue, 10 Jan 2023 13:38:20 +0000	[thread overview]
Message-ID: <bug-107714-4-6P82BcQvCy@http.gcc.gnu.org/bugzilla/> (raw)
In-Reply-To: <bug-107714-4@http.gcc.gnu.org/bugzilla/>

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=107714

--- Comment #7 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-11 branch has been updated by Stam Markianos-Wright
<stammark@gcc.gnu.org>:

https://gcc.gnu.org/g:08842ad274f5e2630994f7c6e70b2d31768107ea

commit r11-10461-g08842ad274f5e2630994f7c6e70b2d31768107ea
Author: Stam Markianos-Wright <stam.markianos-wright@arm.com>
Date:   Fri Dec 30 11:25:22 2022 +0000

    Fix memory constraint on MVE v[ld/st][2/4] instructions [PR107714]

    In the M-Class Arm-ARM:

    https://developer.arm.com/documentation/ddi0553/bu/?lang=en

    these MVE instructions only have '!' writeback variant and at:

    https://gcc.gnu.org/bugzilla/show_bug.cgi?id=107714

    we found that the Um constraint would also allow through a
    register offset writeback, resulting in an assembler error.

    Here I have added a new constraint and predicate for these
    instructions, which (uniquely, AFAICT), only support a `!` writeback
    increment by the data size (inside the compiler this is a POST_INC).

    No regressions in arm-none-eabi with MVE and MVE.FP.

    gcc/ChangeLog:
            PR target/107714
            * config/arm/arm-protos.h (mve_struct_mem_operand): New protoype.
            * config/arm/arm.c (mve_struct_mem_operand): New function.
            * config/arm/constraints.md (Ug): New constraint.
            * config/arm/mve.md (mve_vst4q<mode>): Change constraint.
            (mve_vst2q<mode>): Likewise.
            (mve_vld4q<mode>): Likewise.
            (mve_vld2q<mode>): Likewise.
            * config/arm/predicates.md (mve_struct_operand): New predicate.

    gcc/testsuite/ChangeLog:
            PR target/107714
            * gcc.target/arm/mve/intrinsics/vldst24q_reg_offset.c: New test.

    (cherry picked from commit 4269a6567eb991e6838f40bda5be9e3a7972530c)

  parent reply	other threads:[~2023-01-10 13:38 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-11-16 12:56 [Bug target/107714] New: " kevin.bracey at alifsemi dot com
2022-11-16 15:52 ` [Bug target/107714] " kevin.bracey at alifsemi dot com
2022-11-16 16:00 ` kevin.bracey at alifsemi dot com
2022-11-17 16:43 ` stammark at gcc dot gnu.org
2022-11-21 12:12 ` kevin.bracey at alifsemi dot com
2022-11-21 12:49 ` kevin.bracey at alifsemi dot com
2022-12-09 13:29 ` stammark at gcc dot gnu.org
2022-12-30 11:25 ` cvs-commit at gcc dot gnu.org
2023-01-10 13:38 ` cvs-commit at gcc dot gnu.org [this message]
2023-01-10 13:41 ` cvs-commit at gcc dot gnu.org
2023-01-16 13:22 ` stammark at gcc dot gnu.org

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=bug-107714-4-6P82BcQvCy@http.gcc.gnu.org/bugzilla/ \
    --to=gcc-bugzilla@gcc.gnu.org \
    --cc=gcc-bugs@gcc.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).