Re: C++ PATCH for c++/69657 (abs not inlined)

2016-02-16 Thread Jason Merrill
On 02/16/2016 02:02 PM, Jakub Jelinek wrote: On Tue, Feb 16, 2016 at 02:00:45PM -0500, Jason Merrill wrote: On 02/10/2016 10:31 AM, Jason Merrill wrote: On 02/09/2016 03:29 PM, Rainer Orth wrote: This patch broke Solaris bootstrap (seen on i386-pc-solaris2.12): Fixed by pruning hidden names

Re: C++ PATCH for c++/69657 (abs not inlined)

2016-02-16 Thread Jakub Jelinek
On Tue, Feb 16, 2016 at 02:00:45PM -0500, Jason Merrill wrote: > On 02/10/2016 10:31 AM, Jason Merrill wrote: > >On 02/09/2016 03:29 PM, Rainer Orth wrote: > >>This patch broke Solaris bootstrap (seen on i386-pc-solaris2.12): > > > >Fixed by pruning hidden names from the lookup result in more place

Re: C++ PATCH for c++/69657 (abs not inlined)

2016-02-16 Thread Jason Merrill
On 02/10/2016 10:31 AM, Jason Merrill wrote: On 02/09/2016 03:29 PM, Rainer Orth wrote: This patch broke Solaris bootstrap (seen on i386-pc-solaris2.12): Fixed by pruning hidden names from the lookup result in more places. ...and this broke some dubious code in LLVM. Fixed thus. Tested x86

Re: C++ PATCH for c++/69657 (abs not inlined)

2016-02-10 Thread Jason Merrill
On 02/09/2016 03:29 PM, Rainer Orth wrote: This patch broke Solaris bootstrap (seen on i386-pc-solaris2.12): Fixed by pruning hidden names from the lookup result in more places. Jason commit 09bb9e19d3f284c2f9d8bf57959c99334363c3c9 Author: Jason Merrill Date: Tue Feb 9 16:20:46 2016 -0500

Re: C++ PATCH for c++/69657 (abs not inlined)

2016-02-09 Thread Rainer Orth
Jason Merrill writes: > The issue in this bug was that due to changes in the libstdc++ headers, the > built-in abs declaration was getting hidden by a using-declaration, so that > then when the built-in got an explicit declaration, the original > declaration wasn't there anymore and so the new de