On 09.08.2019. 23:31, Jeff Law wrote:
> On 8/5/19 4:47 AM, Dragan Mladjenovic wrote:
>> From: "Dragan Mladjenovic" <dmladjeno...@wavecomp.com>
>>
>> gcc/ChangeLog:
>>
>> 2019-08-05  Dragan Mladjenovic  <dmladjeno...@wavecomp.com>
>>
>>      * config/mips/linux.h (NEED_INDICATE_EXEC_STACK): Define to
>>      TARGET_SOFT_FLOAT.
>>      * config/mips/mips.c (TARGET_ASM_FILE_END): Define to ...
>>      (mips_asm_file_end): New function. Delegate to
>>      file_end_indicate_exec_stack if NEED_INDICATE_EXEC_STACK is true.
>>      * config/mips/mips.h (NEED_INDICATE_EXEC_STACK): Define to 0.
>>
>> libgcc/ChangeLog:
>>
>> 2019-08-05  Dragan Mladjenovic  <dmladjeno...@wavecomp.com>
>>
>>      * config/mips/gnustack.h: New file.
>>      * config/mips/crti.S: Include gnustack.h.
>>      * config/mips/crtn.S: Likewise.
>>      * config/mips/mips16.S: Likewise.
>>      * config/mips/vr4120-div.S: Likewise.
> Seems reasonable.  What testing has been done for this patch?  I don't
> doubt it works for the MIPS linux targets, I'm more interested in making
> sure it doesn't do the wrong thing for the embedded mips targets.

I've built a cross mips-mti-elf toolchain, albeit with reduced multilib, 
but still there were not .note.GNU-stack in sysroot.
Is this enough?

 >
 > Do you have write access to the repository?
 >

I do not have write access to the repository.

Best regards,
Dragan

Reply via email to