Re: [PATCH 0/2] RISC-V big endian support

2021-02-20 Thread Marcus Comstedt
Hi Kito, Kito Cheng writes: > I tried to run with gcc testsuite on spike + pk, seems got a bunch > extra fail cases, > could you take a look for that? Looking at it now. I'm getting a large number of failures in floating point tests, which turn out to be due to a newlib problem; newlib/libc/

Re: [PATCH 0/2] RISC-V big endian support

2021-02-19 Thread Kito Cheng via Gcc-patches
Hi Marcus: > I could also disable the -mlittle-endian and -mbig-endian options if > the binutils is too old, but it's questionable if getting a "I don't > understand -mbig-endian" from gcc is more useful than getting it from > gas. And it would be more work for the user to fix it since then they

Re: [PATCH 0/2] RISC-V big endian support

2021-01-29 Thread Marcus Comstedt
Hi Kito, Kito Cheng writes: > You can add a check in configure.ac and config.in to detect whether > binutils is supported or not. > here is example, search HAVE_AS_MISA_SPEC in this patch: > https://github.com/gcc-mirror/gcc/commit/4b81528241ca682025d92558ff6aeec91dafdca8 Ok, but I specifical

Re: [PATCH 0/2] RISC-V big endian support

2021-01-29 Thread Kito Cheng via Gcc-patches
Hi Marcus: > * Implicitly pass default endianness as -mbig-endian / -mlittle-endian > to gas, since older binutils targeting little endian will not understand > -mlittle-endian. Instead assume gas has the same default as gcc. > If -mbig-endian or -mlittle-endian is explicitly given, it is p

Re: [PATCH 0/2] RISC-V big endian support

2021-01-26 Thread Kito Cheng via Gcc-patches
Hi Marcus: Let you know reviewing this patch is on my TODO list, and I definitely want this included in GCC 11, but I am working on other urgent work this week, so I might not be able to give any review comment or testing at this moment. Thanks for your awesome work for big-endian support! On Tu

[PATCH 0/2] RISC-V big endian support

2021-01-25 Thread Marcus Comstedt
Greetings. Big endian support for RISC-V was added in binutils 2.36, released yesterday. This patch set adds support to gcc for exploting this mode. Please note that in addition to these patches, an pull of config.sub from upstreams is needed. In adding the big endian support, I had the followi