Re: [PATCH 2/2] gcc/riscv: Add a mechanism to remove some calls to _riscv_save_0

2019-09-06 Thread Jim Wilson
On Sat, Aug 31, 2019 at 1:08 AM Andreas Schwab wrote: > I think the problem is that riscv needs to use t-slibgcc-libgcc so that > libgcc_s.so is a linker script. Yes, thanks, that fixes the problem and works well. I've got a patch using this solution now. Jim

Re: [PATCH v2 4/6] compiler-gcc.h: add asm_inline definition

2019-09-06 Thread Nick Desaulniers
On Fri, Sep 6, 2019 at 3:56 PM Segher Boessenkool wrote: > > On Fri, Sep 06, 2019 at 03:35:02PM -0700, Nick Desaulniers wrote: > > On Fri, Sep 6, 2019 at 3:03 PM Segher Boessenkool > > wrote: > > > And if instead you tested whether the actual feature you need works as > > > you need it to, it wou

[PATCH] RISC-V: Re-enable -msave-restore for shared libraries.

2019-09-06 Thread Jim Wilson
Thanks to Andreas Schwab for the pointer on how to fix this properly. This re-enables -msave-restore for shared libraries, and uses the t-slibgcc-libgcc file to get the save-restore routines included directly in shared libraries so that we don't need to indirect through the PLT to reach them, which

Re: [PATCH v2 4/6] compiler-gcc.h: add asm_inline definition

2019-09-06 Thread Segher Boessenkool
On Fri, Sep 06, 2019 at 04:42:58PM -0700, Nick Desaulniers via gcc-patches wrote: > On Fri, Sep 6, 2019 at 3:56 PM Segher Boessenkool > wrote: > Oh, I misunderstood. I see your point. Define the symbol as a number > for what level of output flags you support (ie. the __cplusplus > macro). That

Re: [PATCH v2 4/6] compiler-gcc.h: add asm_inline definition

2019-09-06 Thread Nick Desaulniers via gcc-patches
On Fri, Sep 6, 2019 at 5:14 PM Segher Boessenkool wrote: > > On Fri, Sep 06, 2019 at 04:42:58PM -0700, Nick Desaulniers via gcc-patches > wrote: > > Just to prove my point about version checks being brittle, it looks > > like Rasmus' version check isn't even right. GCC supported `asm > > inline`

[PATCH] Fix PR 91684

2019-09-06 Thread Bernd Edlinger
Hi, this fixes PR 91684, where an only 4-byte aligned memory is used with movdi, which is formally invalid for strict alignment, but okay for prefer_ldrd_strd targets. Boot-strapped and reg-tested on arm-linux-gnueabihf. Patch was approved via BZ. Applied to trunk. Thanks Bernd. 2019-09-06 Be

Re: [PATCH] Build proper false constant for VECTOR_TYPEs.

2019-09-06 Thread Marc Glisse
On Fri, 6 Sep 2019, Martin Liška wrote: I've been working on transition of cond expressions to match.pd. With my changes I noticed there's one wrong pattern that leads to: Transforming _6 > _7 & _6 < _7 into 0 ... /home/marxin/Programming/gcc/gcc/testsuite/c-c++-common/vector-compare-3.c:20:1:

<    1   2