Hi Alistair, On Sat, Jun 20, 2020 at 1:09 AM Alistair Francis <[email protected]> wrote: > > From: Bin Meng <[email protected]> > > There is no need to have two functions that have exactly the same > codes for 32-bit and 64-bit base CPUs. > > Signed-off-by: Bin Meng <[email protected]> > Reviewed-by: Alistair Francis <[email protected]> > Message-id: [email protected] > Message-Id: <[email protected]>
I noticed that patches from other people than you have the "Message-id" tags, but your patch [1] does not. Is this intentional? (not sure why we need 2 "Message-id" tags here, with one has <> ?) Just want to know what's the best practice here. > Signed-off-by: Alistair Francis <[email protected]> > --- > target/riscv/cpu.c | 18 +++++------------- > 1 file changed, 5 insertions(+), 13 deletions(-) > [1] https://lists.gnu.org/archive/html/qemu-devel/2020-06/msg06208.html Regards, Bin
