This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH] RISC-V: Convert the ADD/ADDI to the compressed MV/LI if RS1 is zero.
- From: Kito Cheng <kito dot cheng at gmail dot com>
- To: Jim Wilson <jimw at sifive dot com>
- Cc: Nelson Chu <nelson dot chu at sifive dot com>, Binutils <binutils at sourceware dot org>, Kito Cheng <kito dot cheng at sifive dot com>
- Date: Thu, 20 Feb 2020 10:47:44 +0800
- Subject: Re: [PATCH] RISC-V: Convert the ADD/ADDI to the compressed MV/LI if RS1 is zero.
- References: <1582076536-10638-1-git-send-email-nelson.chu@sifive.com> <CAFyWVaYQ6H9fp_weDRRASeJWLnEMAUz1KZYTKk7s=kM1Qa5J6A@mail.gmail.com>
Hi Jim:
FYI, I used to using this command to update author of last commit
without change global git setting:
git commit --amend --author="Author Name <email@address.com>"
[1] https://stackoverflow.com/questions/3042437/how-to-change-the-commit-author-for-one-specific-commit
On Thu, Feb 20, 2020 at 7:11 AM Jim Wilson <jimw@sifive.com> wrote:
>
> On Tue, Feb 18, 2020 at 5:42 PM Nelson Chu <nelson.chu@sifive.com> wrote:
> > gas/
> > * testsuite/gas/riscv/c-add-addi.d: New testcase.
> > * testsuite/gas/riscv/c-add-addi.s: Likewise.
> > opcodes/
> > * riscv-opc.c (riscv_opcodes): Convert add/addi to the compressed
> > c.mv/c.li if rs1 is zero.
>
> Looks good to me. Committed.
>
> Though apparently I committed it wrong as I didn't realize that git
> has a way to assign authors to patches separate from the person that
> commits the patch. I just included a changelog line to attribute the
> patch to you. I shall have to try setting the author for the commit
> next time.
>
> Jim