[PATCH] Re: MIPS support for __builtin_flush_icache() breaks -mips16

Sandra Loosemore sandra@codesourcery.com
Mon Jul 30 12:53:00 GMT 2007


David Daney wrote:
> Sandra Loosemore wrote:
>> Now that mainline HEAD is buildable on MIPS again, I've observed that 
>> this patch
>>
>> http://gcc.gnu.org/ml/gcc-patches/2007-07/msg01026.html
>>
>> causes failures with -mips16, e.g. on 
>> gcc.c-torture/compile/20050122-2.c.
> 
> That makes sense to me.  However I am have never tested mips16.
> 
> There are several options:
> 
> 1) Tell me how you configured and tested.  I think one of my boards will 
> do mips16 so I could probably test it if I hacked around a little.
> 
> 2) You could prepare a patch and test with that change as it seems that 
> you are able to do mips16 testing.

Well, the answer to 1) is that I'm building mipsisa32r2-elfoabi and testing 
using the GDB simulator.  But, if you're not set up to do this, 2) is probably 
easier all around, so I built and tested the attached patch overnight, and 
verified that it fixes the MIPS16 regressions.

OK to commit?

-Sandra
-------------- next part --------------
A non-text attachment was scrubbed...
Name: synci.log
Type: text/x-log
Size: 128 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20070730/4077b5a5/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: synci.patch
Type: text/x-patch
Size: 596 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20070730/4077b5a5/attachment-0001.bin>


More information about the Gcc-patches mailing list