Re: [PATCH] SVN_UNALIGNED_ACCESS_IS_OK for ppc

2013-05-22 Thread Ben Reser
On Wed, May 22, 2013 at 12:58 AM, Markus Schaber wrote: > Hmm, when checking those places, I'm not sure whether modern compilers > with their auto-vectorizing optimizers really profit from the hand-optimized > code there, or whether that code might rather trouble the optimizer into > generating su

AW: [PATCH] SVN_UNALIGNED_ACCESS_IS_OK for ppc

2013-05-22 Thread Markus Schaber
Hi, Von: Ben Reser [mailto:b...@reser.org] > On Thu, May 16, 2013 at 2:27 PM, Branko Čibej wrote: > > On 16.05.2013 23:12, Mattias Engdegård wrote: > >> [[[ > >> * subversion/include/svn_types.h: > >> Set SVN_UNALIGNED_ACCESS_IS_OK for PowerPC. > >> ]]] > > > > At first I thought, "that can't b

Re: [PATCH] SVN_UNALIGNED_ACCESS_IS_OK for ppc

2013-05-21 Thread Ben Reser
On Thu, May 16, 2013 at 2:27 PM, Branko Čibej wrote: > On 16.05.2013 23:12, Mattias Engdegård wrote: >> [[[ >> * subversion/include/svn_types.h: >> Set SVN_UNALIGNED_ACCESS_IS_OK for PowerPC. >> ]]] > > At first I thought, "that can't be right!" but it turns out that it is > ... somewhat. Only i

Re: [PATCH] SVN_UNALIGNED_ACCESS_IS_OK for ppc

2013-05-16 Thread Stefan Fuhrmann
As discussed on IRC. Committed as r1483575 with trivial formatting changes. Thanks for the patch! -- Stefan^2. On Thu, May 16, 2013 at 11:12 PM, Mattias Engdegård wrote: > [[[ > * subversion/include/svn_types.**h: > Set SVN_UNALIGNED_ACCESS_IS_OK for PowerPC. > ]]] > -- *Join one of our

Re: [PATCH] SVN_UNALIGNED_ACCESS_IS_OK for ppc

2013-05-16 Thread Branko Čibej
On 16.05.2013 23:12, Mattias Engdegård wrote: > [[[ > * subversion/include/svn_types.h: > Set SVN_UNALIGNED_ACCESS_IS_OK for PowerPC. > ]]] At first I thought, "that can't be right!" but it turns out that it is ... somewhat. Only if we're sure that we use that macro for integer access only. Whic

[PATCH] SVN_UNALIGNED_ACCESS_IS_OK for ppc

2013-05-16 Thread Mattias Engdegård
[[[ * subversion/include/svn_types.h: Set SVN_UNALIGNED_ACCESS_IS_OK for PowerPC. ]]] Index: subversion/include/svn_types.h === --- subversion/include/svn_types.h (revision 1483563) +++ subversion/include/svn_types.h (work