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.

Do you have write access to the repository?

jeff

Reply via email to