Messages in this thread | | | Subject | Re: [PATCH v2 4/8] objtool/LoongArch: Enable orc to be built | From | Tiezhu Yang <> | Date | Sat, 14 Oct 2023 11:40:14 +0800 |
| |
On 10/10/2023 08:52 PM, Huacai Chen wrote: > Hi, Tiezhu, > > On Mon, Oct 9, 2023 at 9:03 PM Tiezhu Yang <yangtiezhu@loongson.cn> wrote: >> >> Implement arch-specific init_orc_entry(), reg_name(), orc_type_name(), >> print_reg() and orc_print_dump(), then set BUILD_ORC as y to build the >> orc related files.
...
>> +#define ORC_REG_SP 2 >> +#define ORC_REG_BP 3 > There is no BP register for LoongArch, so I think all 'BP' should be > 'FP' in this patch.
Makes sense, thank you, will do it.
>> +#define ORC_REG_MAX 4
...
>> +struct orc_entry { >> + s16 sp_offset; >> + s16 bp_offset; >> + s16 ra_offset; >> + unsigned int sp_reg:4; >> + unsigned int bp_reg:4; >> + unsigned int ra_reg:4; >> + unsigned int type:3; >> + unsigned int signal:1; >> +};
At the same time, I will replace bp_offset with fp_offset and replace bp_reg with fp_reg, then modify the related code.
Thanks, Tiezhu
| |