Re: [C++ Patch] PR c++/51188

2011-11-18 Thread Jason Merrill
On 11/18/2011 02:53 PM, Fabien Chêne wrote: I have just committed. Unfortunately, I have mentioned the wrong PR number in the ChangeLog. Consequently, Bugzilla isn't quite happy, shall I adjust the previous commit with the good PR number (51188) ? Yes, do adjust the ChangeLog. I do this period

Re: [C++ Patch] PR c++/51188

2011-11-18 Thread Fabien Chêne
2011/11/18 Jason Merrill : > On 11/18/2011 05:18 AM, Fabien Chêne wrote: >> >> +template<  size_t, size_t>  struct AlignedBuffer; >> +template<  size_t size>  struct AlignedBuffer >> +<  size, 8>  { >> +}; > > This testcase will break on targets where alignof(void*) != 8; just define > the base tem

Re: [C++ Patch] PR c++/51188

2011-11-18 Thread Jason Merrill
On 11/18/2011 05:18 AM, Fabien Chêne wrote: +template< size_t, size_t> struct AlignedBuffer; +template< size_t size> struct AlignedBuffer +< size, 8> { +}; This testcase will break on targets where alignof(void*) != 8; just define the base template and remove the partial specialization.

[C++ Patch] PR c++/51188

2011-11-18 Thread Fabien Chêne
Hi, As discussed in the audit trail, here is a patch that fixes this PR and all its duplicates. Tested x86_64-unknown-linux-gnu without regressions. OK to commit ? gcc/testsuite/ChangeLog 2011-11-18 Fabien Chêne PR c++/51188 * g++.dg/lookup/using46.C: New. * g++.dg/lo