Re: [PATCH] Add "riscv" as an alias for "riscv32"

2018-06-22 Thread Palmer Dabbelt
On Fri, 22 Jun 2018 03:22:57 PDT (-0700), rjo...@redhat.com wrote: 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 >>

Re: [PATCH] Add "riscv" as an alias for "riscv32"

2018-06-22 Thread John Ericson
DJ's and Karsten's concerns match my config-for-target vs config-for-toolchain distinction. arm*- riscv-* and x86-* do make sense as compiler prefixes that indicate toolchain support for a set of targets. But when we think of a toolchain + a set of standard libraries (without multilib), only a

Re: [PATCH] Add "riscv" as an alias for "riscv32"

2018-06-22 Thread Palmer Dabbelt
On Thu, 21 Jun 2018 18:17:45 PDT (-0700), d...@redhat.com wrote: If by this patch you mean that "riscv-unknown-linux-gnu" means EITHER riscv32 (when on a 32-bit os) OR riscv64 (when on a 64-bit os), then I would prefer *against* this, as it would mean that config.guess would return an ambigious v

Re: [PATCH] Add "riscv" as an alias for "riscv32"

2018-06-22 Thread Manuel A. Fernandez Montecelo
2018-06-22 3:17 GMT+02:00 DJ Delorie : > > If by this patch you mean that "riscv-unknown-linux-gnu" means EITHER > riscv32 (when on a 32-bit os) OR riscv64 (when on a 64-bit os), then I > would prefer *against* this, as it would mean that config.guess would > return an ambigious value - there is no

Re: [PATCH] Add "riscv" as an alias for "riscv32"

2018-06-22 Thread Richard W.M. Jones
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 > >> > >>* config.sub: Add "riscv-*" as an alias for "riscv3

Re: [PATCH] Add "riscv" as an alias for "riscv32"

2018-06-22 Thread Ben Elliston
On Fri, Jun 22, 2018 at 08:51:23AM +0100, Richard W.M. Jones wrote: > Is there some more context to this discussion I can read? http://lists.gnu.org/archive/html/config-patches/2018-06/index.html Cheers, Ben signature.asc Description: PGP signature _

Re: [PATCH] Add "riscv" as an alias for "riscv32"

2018-06-22 Thread Richard W.M. Jones
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 > >> > >>* config.sub: Add "riscv-*" as an alias for "riscv3