mbox series

[0/1] RISC-V: Handle 'Zca' extension on ".option arch"

Message ID cover.1690250175.git.research_trasio@irq.a4lg.com
Headers
Series RISC-V: Handle 'Zca' extension on ".option arch" |

Message

Tsukasa OI July 25, 2023, 1:56 a.m. UTC 
  Hi,

While I'm testing the PATCH v2 of the 'Zihintntl', I found a bug in GAS.

(Normally I wouldn't attach a cover letter for the patch set that simple,
I felt it's needed for a little context; PATCH v2 of 'Zihintntl' will need
this bug fix.)

While the 'Zca' extension is the new base for compressed instructions,
it was not correctly handled in the ".option arch" handling.  This patch
set fixes this issue.  After grepping "c", I think it's the last code to
test.

Thanks,
Tsukasa




Tsukasa OI (1):
  RISC-V: Enable RVC on ".option arch, +zca" etc.

 gas/config/tc-riscv.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)


base-commit: 3dfee72010248aca5932a27b4c34d3b79ab9f057

Comments

Nelson Chu July 25, 2023, 2:21 a.m. UTC | #1 
Thanks, I did forget this, it looks good.

Nelson

On Tue, Jul 25, 2023 at 9:56 AM Tsukasa OI <research_trasio@irq.a4lg.com>
wrote:

> Hi,
>
> While I'm testing the PATCH v2 of the 'Zihintntl', I found a bug in GAS.
>
> (Normally I wouldn't attach a cover letter for the patch set that simple,
> I felt it's needed for a little context; PATCH v2 of 'Zihintntl' will need
> this bug fix.)
>
> While the 'Zca' extension is the new base for compressed instructions,
> it was not correctly handled in the ".option arch" handling.  This patch
> set fixes this issue.  After grepping "c", I think it's the last code to
> test.
>
> Thanks,
> Tsukasa
>
>
>
>
> Tsukasa OI (1):
>   RISC-V: Enable RVC on ".option arch, +zca" etc.
>
>  gas/config/tc-riscv.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
>
> base-commit: 3dfee72010248aca5932a27b4c34d3b79ab9f057
> --
> 2.41.0
>
>
Tsukasa OI July 25, 2023, 2:23 a.m. UTC | #2 
On 2023/07/25 11:21, Nelson Chu wrote:
> Thanks, I did forget this, it looks good.
> 
> Nelson

Thanks, committing.

Best,
Tsukasa

> 
> On Tue, Jul 25, 2023 at 9:56 AM Tsukasa OI <research_trasio@irq.a4lg.com
> <mailto:research_trasio@irq.a4lg.com>> wrote:
> 
>     Hi,
> 
>     While I'm testing the PATCH v2 of the 'Zihintntl', I found a bug in GAS.
> 
>     (Normally I wouldn't attach a cover letter for the patch set that
>     simple,
>     I felt it's needed for a little context; PATCH v2 of 'Zihintntl'
>     will need
>     this bug fix.)
> 
>     While the 'Zca' extension is the new base for compressed instructions,
>     it was not correctly handled in the ".option arch" handling.  This patch
>     set fixes this issue.  After grepping "c", I think it's the last code to
>     test.
> 
>     Thanks,
>     Tsukasa
> 
> 
> 
> 
>     Tsukasa OI (1):
>       RISC-V: Enable RVC on ".option arch, +zca" etc.
> 
>      gas/config/tc-riscv.c | 3 ++-
>      1 file changed, 2 insertions(+), 1 deletion(-)
> 
> 
>     base-commit: 3dfee72010248aca5932a27b4c34d3b79ab9f057
>     -- 
>     2.41.0
>