[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Add "riscv" as an alias for "riscv32"
From: |
Richard W.M. Jones |
Subject: |
Re: [PATCH] Add "riscv" as an alias for "riscv32" |
Date: |
Fri, 22 Jun 2018 08:51:23 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Thu, Jun 21, 2018 at 05:25:59PM -0700, Palmer Dabbelt wrote:
> On Thu, 21 Jun 2018 16:16:51 PDT (-0700), address@hidden wrote:
> >On Wed, Jun 20, 2018 at 10:24:41AM -0700, Palmer Dabbelt wrote:
> >
> >>2018-06-11 Palmer Dabbelt <address@hidden>
> >>
> >> * config.sub: Add "riscv-*" as an alias for "riscv32-*".
> >> * testsuite/config-sub.data: Add tests for the "riscv-*" alias.
> >
> >In light of the discussion, I think this patch is OK. Do we want to
> >change config.guess to always emit riscv for the native configuration,
> >like so?
> >
> >diff --git a/config.guess b/config.guess
> >index 883a671..c501b8a 100755
> >--- a/config.guess
> >+++ b/config.guess
> >@@ -1025,7 +1025,7 @@ EOF
> > echo powerpcle-unknown-linux-"$LIBC"
> > exit ;;
> > riscv32:Linux:*:* | riscv64:Linux:*:*)
> >- echo "$UNAME_MACHINE"-unknown-linux-"$LIBC"
> >+ echo riscv-unknown-linux-"$LIBC"
> > exit ;;
> > s390:Linux:*:* | s390x:Linux:*:*)
> > echo "$UNAME_MACHINE"-ibm-linux-"$LIBC"
>
> That sounds reasonable to me. The only wrinkle I can see here is
> that if "riscv-*" maps to "riscv32-*", you're on a 64-bit system,
> and we assume that compilers running on RISC-V machines default to a
> native target, does that mean that the logical "riscv32-*" is a
> 64-bit compiler?
>
> It's not a big deal as configure scripts should just be matching
> "riscv*-*" anyway, but maybe it's a problem for native userspace.
> I'm adding some native distro people to see if anyone has any
> opinions, as I'm far from an expert on this sort of stuff.
Is there some more context to this discussion I can read?
Rich.
--
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
virt-top is 'top' for virtual machines. Tiny program with many
powerful monitoring features, net stats, disk stats, logging, etc.
http://people.redhat.com/~rjones/virt-top
- Re: [PATCH] Add "riscv" as an alias for "riscv32", (continued)
Re: [PATCH] Add "riscv" as an alias for "riscv32", Ben Elliston, 2018/06/21
Re: [PATCH] Add "riscv" as an alias for "riscv32", Richard W.M. Jones, 2018/06/22
Re: [PATCH] Add "riscv" as an alias for "riscv32", Palmer Dabbelt, 2018/06/22
Message not availableRe: [PATCH] Add "riscv" as an alias for "riscv32", John Ericson, 2018/06/22
Re: [PATCH] Add "riscv" as an alias for "riscv32", Liviu Ionescu, 2018/06/21