Re: [PATCH v2] [MIPS]: add .module mipsREV to all output asm file

2021-09-02 Thread Jeff Law via Gcc-patches
On 8/28/2021 1:08 AM, Xi Ruoyao wrote: On Fri, 2021-08-27 at 15:36 -0600, Jeff Law wrote: It's easier when someone has to debug the code later. enums show up in debug output by default, while #defines do not. switch (mips_isa)    { case MIPS_ISA_MIPS1: return "mips1"; // ...  

Re: [PATCH v2] MIPS: add .module mipsREV to all output asm file

2021-08-29 Thread YunQiang Su
Please ignore this post, as I mistakenly used -1 instead of -2. YunQiang Su 于2021年8月30日周一 上午10:57写道: > > Currently, the asm output file for MIPS has no rev info. > It can make some trouble, for example: > assembler is mips1 by default, > gcc is fpxx by default. > To assemble the output of gcc

[PATCH v2] MIPS: add .module mipsREV to all output asm file

2021-08-29 Thread YunQiang Su
Currently, the asm output file for MIPS has no rev info. It can make some trouble, for example: assembler is mips1 by default, gcc is fpxx by default. To assemble the output of gcc -S, we have to pass -mips2 to assembler. gcc/ChangeLog: * gcc/config/mips/mips.c (mips_module_isa_name): New

Re: [PATCH v2] [MIPS]: add .module mipsREV to all output asm file

2021-08-28 Thread YunQiang Su
Xi Ruoyao via Gcc-patches 于2021年8月28日周六 下午3:09写道: > > On Fri, 2021-08-27 at 15:36 -0600, Jeff Law wrote: > > > It's easier when someone has to debug the code later. > > enums show up in debug output by default, while #defines do not. > > > > > > > switch (mips_isa) > > >{ > > > case MIPS_

Re: [PATCH v2] [MIPS]: add .module mipsREV to all output asm file

2021-08-28 Thread Xi Ruoyao via Gcc-patches
On Fri, 2021-08-27 at 15:36 -0600, Jeff Law wrote: > It's easier when someone has to debug the code later.  > enums show up in debug output by default, while #defines do not. > > > > switch (mips_isa) > >    { > > case MIPS_ISA_MIPS1: return "mips1"; > > // ... > >    } > > > > It lo

Re: [PATCH v2] [MIPS]: add .module mipsREV to all output asm file

2021-08-27 Thread Jeff Law via Gcc-patches
On 8/26/2021 11:26 PM, Xi Ruoyao via Gcc-patches wrote: On Fri, 2021-08-27 at 13:11 +0800, YunQiang Su wrote: 在 2021/6/18 11:29, YunQiang Su 写道: Currently, the asm output file for MIPS has no rev info. It can make some trouble, for example:    assembler is mips1 by default,    gcc is fpxx

Re: [PATCH v2] [MIPS]: add .module mipsREV to all output asm file

2021-08-26 Thread Xi Ruoyao via Gcc-patches
On Fri, 2021-08-27 at 13:11 +0800, YunQiang Su wrote: > 在 2021/6/18 11:29, YunQiang Su 写道: > > Currently, the asm output file for MIPS has no rev info. > > It can make some trouble, for example: > >    assembler is mips1 by default, > >    gcc is fpxx by default. > > To assemble the output of gcc -

Re: [PATCH v2] [MIPS]: add .module mipsREV to all output asm file

2021-08-26 Thread YunQiang Su
在 2021/6/18 11:29, YunQiang Su 写道: Currently, the asm output file for MIPS has no rev info. It can make some trouble, for example: assembler is mips1 by default, gcc is fpxx by default. To assemble the output of gcc -S, we have to pass -mips2 to assembler. gcc/ChangeLog: * gcc/config

[PATCH v2] [MIPS]: add .module mipsREV to all output asm file

2021-06-17 Thread YunQiang Su
Currently, the asm output file for MIPS has no rev info. It can make some trouble, for example: assembler is mips1 by default, gcc is fpxx by default. To assemble the output of gcc -S, we have to pass -mips2 to assembler. gcc/ChangeLog: * gcc/config/mips/mips.c (mips_module_isa_name): New