Skip to content

Commit 8c1a5f9

Browse files
author
svn-role
committed
Merge r1924100 from trunk:
* r1924100 Deprecate and disable the SVN_UNALIGNED_ACCESS_IS_OK macro. Justification: Prevent future problems on newer compilers on higher optimization settings. Votes: +1: dsahlberg, brane, jamessan git-svn-id: https://svn.apache.org/repos/asf/subversion/branches/1.14.x@1924794 13f79535-47bb-0310-9956-ffa450edef68
1 parent 91c65fc commit 8c1a5f9

File tree

3 files changed

+8
-19
lines changed

3 files changed

+8
-19
lines changed

STATUS

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -36,14 +36,6 @@ Veto-blocked changes:
3636
Approved changes:
3737
=================
3838

39-
* r1924100
40-
Deprecate and disable the SVN_UNALIGNED_ACCESS_IS_OK macro.
41-
Justification:
42-
Prevent future problems on newer compilers on higher optimization
43-
settings.
44-
Votes:
45-
+1: dsahlberg, brane, jamessan
46-
4739
* r1924143
4840
Remove optimized unaligned-access code paths.
4941
Justification:
@@ -58,4 +50,3 @@ Approved changes:
5850
Allow swig-rb with x64-mswin64 and gcc 15
5951
Votes:
6052
+1: jun66j5, jamessan
61-

notes/knobs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ SUFFIX_LINES_TO_KEEP
3838
SVN_FS_FS_DEFAULT_MAX_FILES_PER_DIR
3939
SVN_FS_FS_MAX_LINEAR_DELTIFICATION
4040
SVN_FS_FS_MAX_DELTIFICATION_WALK
41-
SVN_UNALIGNED_ACCESS_IS_OK
41+
SVN_UNALIGNED_ACCESS_IS_OK (deprecated)
4242

4343
2.2 Features
4444

@@ -164,12 +164,14 @@ SVN_I_LIKE_LATENCY_SO_IGNORE_HTTPV2
164164

165165
3.8 SVN_UNALIGNED_ACCESS_IS_OK
166166

167+
DEPRECATED, unused. The rest of this entry is historical.
168+
167169
Scope: (global)
168170
Purpose: enable data accesss optimizations.
169171
If your target CPU supports unaligned memory access without
170172
significant performance penalties, you should enable this
171173
optimization as it allows for processing 4 or 8 bytes at
172-
once instead of just one byte at a time.
174+
once instead of just one byte at a time.
173175
Range: 0 1
174176
Default: platform dependant (see svn_types.h)
175177
Suggested: 0

subversion/include/svn_types.h

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -124,16 +124,12 @@ extern "C" {
124124
* if unaligned access is supported for integers.
125125
*
126126
* @since New in 1.7.
127+
* @deprecated Provided for backward compatibility with the 1.14 API.
127128
*/
128-
#ifndef SVN_UNALIGNED_ACCESS_IS_OK
129-
# if defined(_M_IX86) || defined(i386) \
130-
|| defined(_M_X64) || defined(__x86_64) \
131-
|| defined(__powerpc__) || defined(__ppc__)
132-
# define SVN_UNALIGNED_ACCESS_IS_OK 1
133-
# else
134-
# define SVN_UNALIGNED_ACCESS_IS_OK 0
135-
# endif
129+
#ifdef SVN_UNALIGNED_ACCESS_IS_OK
130+
# undef SVN_UNALIGNED_ACCESS_IS_OK
136131
#endif
132+
#define SVN_UNALIGNED_ACCESS_IS_OK 0
137133

138134

139135

0 commit comments

Comments
 (0)