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