This is the mail archive of the
mailing list for the GCC project.
Re: [PATCH, rs6000] Fix PR target/71733, ICE with -mcpu=power9 -mno-vsx
- From: David Edelsohn <dje dot gcc at gmail dot com>
- To: Peter Bergner <bergner at vnet dot ibm dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Segher Boessenkool <segher at kernel dot crashing dot org>, Bill Schmidt <wschmidt at linux dot vnet dot ibm dot com>, Michael Meissner <meissner at linux dot vnet dot ibm dot com>, Aaron Sawdey <acsawdey at linux dot vnet dot ibm dot com>
- Date: Wed, 6 Jul 2016 13:53:59 -0400
- Subject: Re: [PATCH, rs6000] Fix PR target/71733, ICE with -mcpu=power9 -mno-vsx
- Authentication-results: sourceware.org; auth=none
- References: <email@example.com>
On Tue, Jul 5, 2016 at 10:26 PM, Peter Bergner <firstname.lastname@example.org> wrote:
> The following patch fixes a bug where we do not disable POWER9 vector dform
> addressing when we compile for POWER9 but without VSX support. This manifested
> itself with us trying to use dform addressing with altivec loads/stores
> which is illegal, leading to an ICE.
DFORM definitely should be disabled without VSX, but the patch seems
incomplete. If VSX and DFORM are enabled, and GCC chooses an Altivec
instruction alternative in a pattern, what is to prevent the
generation of a DFORM address?