Re: [C++ PATCH] namespace bindings

2017-05-08 Thread Nathan Sidwell
On 05/08/2017 11:17 AM, Andreas Schwab wrote: g++.dg/cpp0x/defaulted34.C has the same problem. I'd managed to flub my xcompiler build. The code path for fn-descriptor-using targets is different, and I'd missed that the shadowed outer 'fn' was expected to have been initialized in that case.

Re: [C++ PATCH] namespace bindings

2017-05-08 Thread Nathan Sidwell
On 05/08/2017 08:56 AM, Jason Merrill wrote: On Fri, May 5, 2017 at 4:07 PM, Nathan Sidwell wrote: This cleanup patch kills IDENTIFIER_NAMESPACE_VALUE and replaces {get,set}_namespace_binding with get_namespace_value and set_global_value respectively. I'd prefer to stick with the _binding nam

Re: [C++ PATCH] namespace bindings

2017-05-08 Thread Andreas Schwab
On Mai 08 2017, Nathan Sidwell wrote: > On 05/08/2017 08:07 AM, Andreas Schwab wrote: >> On Mai 05 2017, Nathan Sidwell wrote: >> >>> This cleanup patch kills IDENTIFIER_NAMESPACE_VALUE and replaces >>> {get,set}_namespace_binding with get_namespace_value and set_global_value >>> respectively. >

Re: [C++ PATCH] namespace bindings

2017-05-08 Thread Nathan Sidwell
On 05/08/2017 08:07 AM, Andreas Schwab wrote: On Mai 05 2017, Nathan Sidwell wrote: This cleanup patch kills IDENTIFIER_NAMESPACE_VALUE and replaces {get,set}_namespace_binding with get_namespace_value and set_global_value respectively. I'm getting this regression on ia64: FAIL: g++.dg/temp

Re: [C++ PATCH] namespace bindings

2017-05-08 Thread Nathan Sidwell
On 05/08/2017 08:56 AM, Jason Merrill wrote: On Fri, May 5, 2017 at 4:07 PM, Nathan Sidwell wrote: This cleanup patch kills IDENTIFIER_NAMESPACE_VALUE and replaces {get,set}_namespace_binding with get_namespace_value and set_global_value respectively. I'd prefer to stick with the _binding nam

Re: [C++ PATCH] namespace bindings

2017-05-08 Thread Nathan Sidwell
On 05/08/2017 08:07 AM, Andreas Schwab wrote: On Mai 05 2017, Nathan Sidwell wrote: This cleanup patch kills IDENTIFIER_NAMESPACE_VALUE and replaces {get,set}_namespace_binding with get_namespace_value and set_global_value respectively. I'm getting this regression on ia64: Sorry, I'll tak

Re: [C++ PATCH] namespace bindings

2017-05-08 Thread Jason Merrill via gcc-patches
On Fri, May 5, 2017 at 4:07 PM, Nathan Sidwell wrote: > This cleanup patch kills IDENTIFIER_NAMESPACE_VALUE and replaces > {get,set}_namespace_binding with get_namespace_value and set_global_value > respectively. I'd prefer to stick with the _binding naming, as "value" is more ambiguous. Jason

Re: [C++ PATCH] namespace bindings

2017-05-08 Thread Andreas Schwab
On Mai 05 2017, Nathan Sidwell wrote: > This cleanup patch kills IDENTIFIER_NAMESPACE_VALUE and replaces > {get,set}_namespace_binding with get_namespace_value and set_global_value > respectively. I'm getting this regression on ia64: FAIL: g++.dg/template/virtual3.C -std=c++11 (test for excess