Re: [PATCH] New lang hook

2017-11-18 Thread Andreas Schwab
FAIL: g++.dg/pr82836.C (test for excess errors) Excess errors: /opt/gcc/gcc-20171118/gcc/testsuite/g++.dg/pr82836.C:13:3: error: '__float128' does not name a type; did you mean '__Float16x8_t'? Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 0

Re: [PATCH] New lang hook

2017-11-16 Thread Nathan Sidwell
On 11/16/2017 07:03 AM, Richard Biener wrote: Looks reasonable apart from + /* Overwrite the DECL_ASSEMBLER_NAME for a node. The name is being + changed (including to or from NULL_TREE). */ which suggests the default implementation of set_decl_assembler_name would call this hook (which

Re: [PATCH] New lang hook

2017-11-16 Thread Richard Biener
On Tue, Nov 14, 2017 at 1:31 PM, Nathan Sidwell wrote: > This patch addresses c++/82836 & c++/82737. The root cause was a bad > assumption I made when moving the mangling alias machinery to its own hash > table. > > I had thought that once we SET_DECL_ASSEMBLER_NAME, it never becomes unset > (or

[PATCH] New lang hook

2017-11-14 Thread Nathan Sidwell
This patch addresses c++/82836 & c++/82737. The root cause was a bad assumption I made when moving the mangling alias machinery to its own hash table. I had thought that once we SET_DECL_ASSEMBLER_NAME, it never becomes unset (or changed). That is false. There are paths in the compiler tha