This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch, mips] Fix for PR 54619, GCC aborting with -O -mips16
- From: Richard Sandiford <rdsandiford at googlemail dot com>
- To: Tom de Vries <Tom_deVries at mentor dot com>
- Cc: Zdenek Dvorak <rakdver at kam dot mff dot cuni dot cz>, Sandra Loosemore <sandra at codesourcery dot com>, <gcc-patches at gcc dot gnu dot org>, Steve Ellcey <sellcey at mips dot com>
- Date: Mon, 19 Nov 2012 12:01:03 +0000
- Subject: Re: [patch, mips] Fix for PR 54619, GCC aborting with -O -mips16
- References: <505BC593.9000708@codesourcery.com> <50A41B50.5030907@mentor.com>
Tom de Vries <Tom_deVries@mentor.com> writes:
> On 21/09/12 03:40, Sandra Loosemore wrote:
>> Re:
>>
>>> I think tree-ssa-loop-ivopts is simply
>>> asking for the wrong thing, and needs to be changed. As I say,
>>> Sandra had some fixes in this area.
>>
>> This patch:
>>
>> http://gcc.gnu.org/ml/gcc-patches/2012-06/msg00319.html
>>
>> Sadly, that patch has fallen off the bottom of my priority list (some
>> legal wrangling halted all work on the Hexagon port that was the
>> motivating example for it), and meanwhile it's become bit-rotten due to
>> Bill Schmidt's work that touched on the ivopts code as well. If someone
>> else would like to "adopt" this patch, please do!
>>
>> I agree that it is just plain dumb for ivopts to be trying to precompute
>> address costs for modes that are not even valid.
>>
>
> Zdenek,
>
> this patch prevents ivopts from passing VOIDmode to address_cost.
>
> The patch fixes a buildbreaker in newlib for mips16. I've confirmed that I can
> do a full mips build with newlib with this patch.
Many (belated) thanks for doing this. It kept falling off my radar...
Richard