This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [EXTERNAL]Re: [PATCH 1/2][MIPS] Emit .note.GNU-stack for soft-float linux targets.


On 09.08.2019. 23:31, Jeff Law wrote:
> On 8/5/19 4:47 AM, Dragan Mladjenovic wrote:
>> From: "Dragan Mladjenovic" <dmladjenovic@wavecomp.com>
>>
>> gcc/ChangeLog:
>>
>> 2019-08-05  Dragan Mladjenovic  <dmladjenovic@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  <dmladjenovic@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

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]