Re: [PATCH][Aarch64] Add vectorized mersenne twister

2017-06-06 Thread Jonathan Wakely
On 06/06/17 12:41 +0100, Charles Baylis wrote: On 6 June 2017 at 11:07, James Greenhalgh wrote: If we don't mind that, and instead take a GCC-centric view, we could avoid namespace polution by using the GCC-internal names for the intrinsics (__builtin_aarch64_...). As those names don't form a

Re: [PATCH][Aarch64] Add vectorized mersenne twister

2017-06-06 Thread Charles Baylis
On 6 June 2017 at 11:07, James Greenhalgh wrote: > If we don't mind that, and instead take a GCC-centric view, we could avoid > namespace polution by using the GCC-internal names for the intrinsics > (__builtin_aarch64_...). As those names don't form a guaranteed interface, > that would tie us to

Re: [PATCH][Aarch64] Add vectorized mersenne twister

2017-06-06 Thread Jonathan Wakely
On 06/06/17 12:33 +0100, James Greenhalgh wrote: On Tue, Jun 06, 2017 at 11:47:45AM +0100, Jonathan Wakely wrote: On 06/06/17 11:23 +0100, Jonathan Wakely wrote: >On 06/06/17 11:07 +0100, James Greenhalgh wrote: >>On Fri, Jun 02, 2017 at 07:13:17PM +0100, Jonathan Wakely wrote: >>>On 02/06/17 19

Re: [PATCH][Aarch64] Add vectorized mersenne twister

2017-06-06 Thread James Greenhalgh
On Tue, Jun 06, 2017 at 11:47:45AM +0100, Jonathan Wakely wrote: > On 06/06/17 11:23 +0100, Jonathan Wakely wrote: > >On 06/06/17 11:07 +0100, James Greenhalgh wrote: > >>On Fri, Jun 02, 2017 at 07:13:17PM +0100, Jonathan Wakely wrote: > >>>On 02/06/17 19:19 +0200, Ulrich Drepper wrote: > On Fr

Re: [PATCH][Aarch64] Add vectorized mersenne twister

2017-06-06 Thread Ulrich Drepper
On Tue, Jun 6, 2017 at 12:07 PM, James Greenhalgh wrote: > We're a good number of years late to do that without causing some pain. Well, it's pain for those who deserve it. Who thought it to be a smart idea to pollute the global namespace? It's a one-time deal. > So we have a few solutions to

Re: [PATCH][Aarch64] Add vectorized mersenne twister

2017-06-06 Thread Jonathan Wakely
On 06/06/17 11:23 +0100, Jonathan Wakely wrote: On 06/06/17 11:07 +0100, James Greenhalgh wrote: On Fri, Jun 02, 2017 at 07:13:17PM +0100, Jonathan Wakely wrote: On 02/06/17 19:19 +0200, Ulrich Drepper wrote: On Fri, Jun 2, 2017 at 5:46 PM, Michael Collison wrote: This implementation include

Re: [PATCH][Aarch64] Add vectorized mersenne twister

2017-06-06 Thread Jonathan Wakely
On 06/06/17 11:07 +0100, James Greenhalgh wrote: On Fri, Jun 02, 2017 at 07:13:17PM +0100, Jonathan Wakely wrote: On 02/06/17 19:19 +0200, Ulrich Drepper wrote: >On Fri, Jun 2, 2017 at 5:46 PM, Michael Collison > wrote: >>This implementation includes "arm_neon.h" when including the optimized .

Re: [PATCH][Aarch64] Add vectorized mersenne twister

2017-06-06 Thread James Greenhalgh
On Fri, Jun 02, 2017 at 07:13:17PM +0100, Jonathan Wakely wrote: > On 02/06/17 19:19 +0200, Ulrich Drepper wrote: > >On Fri, Jun 2, 2017 at 5:46 PM, Michael Collison > > wrote: > >>This implementation includes "arm_neon.h" when including the optimized > >>. This has the effect of polluting the gl

Re: Fw: [PATCH][Aarch64] Add vectorized mersenne twister

2017-06-02 Thread Jonathan Wakely
On 02/06/17 15:29 +0100, James Greenhalgh wrote: Hi Michael, I think the correct list for this patch would be libstd...@gcc.gnu.org (I'm sure someone there will correct me if I'm wrong). Thanks, James, that's right. All libstdc++ patches should go to both gcc-patches and this list. I'm inte

[PATCH][Aarch64] Add vectorized mersenne twister

2017-06-01 Thread Michael Collison
This patch adds an vectorized implementation of the mersenne twister random number generator. This implementation is approximately 2.6 times faster than the non-vectorized implementation. This implementation includes "arm_neon.h" when including the optimized . This has the effect of polluting