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.

Thanks!

_______________________________________________
config-patches mailing list
config-patches@gnu.org
https://lists.gnu.org/mailman/listinfo/config-patches

Reply via email to