[PATCH v4] RISC-V: Gate opcode tables by enum rather than string
Kito Cheng
kito.cheng@gmail.com
Wed Sep 4 01:38:00 GMT 2019
Hi Jim:
I think commit after branch would be better, to prevent any accident
we didn't catch,
and this patch is more like code refinement/refactor, it didn't change
behavior or fixing bug,
so it should not urgent to merge.
:)
On Wed, Sep 4, 2019 at 9:02 AM Jim Wilson <jimw@sifive.com> wrote:
>
> On Mon, Sep 2, 2019 at 11:14 PM Kito Cheng <kito.cheng@gmail.com> wrote:
> > I've tested that with regression in riscv-gnu-toolchain, LGTM :)
>
> Thanks for testing it. This looks good to me too. We lost the
> ChangeLog entry, and I noticed two very minor comment typos I missed
> on the earlier review, "atleast" should be "at least" and "belongs.
> Used" should be "belongs. Used" as coding conventions call for two
> spaces after a period. I'm happy to fix this stuff myself and commit
> the patch.
>
> There is another issue here though which is that the binutils-2.33
> branch is scheduled for Sept 7, and this isn't a bug fix, it is
> preparation for adding new features. Do we want to include this now?
> Or should we wait until after the branch?
>
> Jim
More information about the Binutils
mailing list