On Thu, Jun 21, 2018 at 05:25:59PM -0700, Palmer Dabbelt wrote: > On Thu, 21 Jun 2018 16:16:51 PDT (-0700), b...@air.net.au wrote: > >On Wed, Jun 20, 2018 at 10:24:41AM -0700, Palmer Dabbelt wrote: > > > >>2018-06-11 Palmer Dabbelt <pal...@sifive.com> > >> > >> * 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 _______________________________________________ config-patches mailing list config-patches@gnu.org https://lists.gnu.org/mailman/listinfo/config-patches
