This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH], Pr 71667, Fix PowerPC ISA 3.0 DImode Altivec load/store
- From: Segher Boessenkool <segher at kernel dot crashing dot org>
- To: Michael Meissner <meissner at linux dot vnet dot ibm dot com>, gcc-patches at gcc dot gnu dot org, David Edelsohn <dje dot gcc at gmail dot com>, Bill Schmidt <wschmidt at linux dot vnet dot ibm dot com>
- Date: Tue, 28 Jun 2016 12:49:12 -0500
- Subject: Re: [PATCH], Pr 71667, Fix PowerPC ISA 3.0 DImode Altivec load/store
- Authentication-results: sourceware.org; auth=none
- References: <20160628000819 dot GA12447 at ibm-tiger dot the-meissners dot org> <20160628023716 dot GG27655 at gate dot crashing dot org> <20160628172745 dot GA17897 at ibm-tiger dot the-meissners dot org>
On Tue, Jun 28, 2016 at 01:27:45PM -0400, Michael Meissner wrote:
<snip>
Thanks for the explanation.
> I think the thing to do is create yet another memory constraint, that is just
> an offsetable address, with the bottom 2 bits 0, and no PRE_MODIFY, etc.
That sounds best yes. The current patch seemed fragile and a bit
confusing / surprising to me, but you now found an actual problem as
well.
> > 20599 lines, can you minimize this a bit? If not, maybe we should just
> > do without testcase here.
>
> I doubt I could minimize it, since it is only this one source so far that has
> shown to be failure. I can delete it if you prefer.
Huge testcases for simple problems does not really scale.
Segher