[PATCH][MIPS] mwarn-framesize= option

Mark Mitchell mark@codesourcery.com
Fri Jun 13 20:04:00 GMT 2008


Richard Sandiford wrote:

>> Since we are in stage 1, this would be the time to get this in.  We 
>> would then not have two different ways to specify very nearly the same 
>> thing.  It would also save us the trouble of deprecating and removing a 
>> mips specific version in the future.
> 
> Agreed 100%.  FWIW, this was also what Seongbae and I thought when
> the -Wframe-larger-than patch originally went in:
> 
>     http://gcc.gnu.org/ml/gcc-patches/2008-02/msg00642.html

It seems a bit unfortunate to me that we're using Maxim's patch to try 
to force someone to implement this bit of infrastructure.  If we wanted 
the target hook, I think we should have asked the original submitter of 
the patch you mention to implement it at the time as a condition of 
putting it in.

However, I guess we can ask politely now...

Seongbae, is adding some hook mechanism so that back-ends can refine the 
frame-size check something that you would be able to take on?

Thanks,

-- 
Mark Mitchell
CodeSourcery
mark@codesourcery.com
(650) 331-3385 x713



More information about the Gcc-patches mailing list