Skip to content

Commit 154a132

Browse files
committed
size assert no work, try other with cast
1 parent 0784d19 commit 154a132

File tree

3 files changed

+18
-18
lines changed

3 files changed

+18
-18
lines changed

Include/cpython/pyatomic_gcc.h

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -608,9 +608,9 @@ _Py_atomic_load_ssize_acquire(const Py_ssize_t *obj)
608608
static inline void *
609609
_Py_atomic_memcpy_ptr_store_relaxed(void *dest, void *src, size_t n)
610610
{
611-
// assert(((uintptr_t)dest & (sizeof (void *) - 1)) == 0);
612-
// assert(((uintptr_t)src & (sizeof (void *) - 1)) == 0);
613-
assert(n % (size_t)sizeof(void *) == 0);
611+
assert(((uintptr_t)dest & (uintptr_t)(sizeof (void *) - 1)) == 0);
612+
assert(((uintptr_t)src & (uintptr_t)(sizeof (void *) - 1)) == 0);
613+
// assert(n % sizeof(void *) == 0);
614614

615615
// memcpy(dest, src, n);
616616

@@ -631,9 +631,9 @@ _Py_atomic_memcpy_ptr_store_relaxed(void *dest, void *src, size_t n)
631631
static inline void *
632632
_Py_atomic_memmove_ptr_store_relaxed(void *dest, void *src, size_t n)
633633
{
634-
// assert(((uintptr_t)dest & (sizeof (void *) - 1)) == 0);
635-
// assert(((uintptr_t)src & (sizeof (void *) - 1)) == 0);
636-
assert(n % (size_t)sizeof(void *) == 0);
634+
assert(((uintptr_t)dest & (uintptr_t)(sizeof (void *) - 1)) == 0);
635+
assert(((uintptr_t)src & (uintptr_t)(sizeof (void *) - 1)) == 0);
636+
// assert(n % sizeof(void *) == 0);
637637

638638
// memmove(dest, src, n);
639639

Include/cpython/pyatomic_msc.h

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1162,9 +1162,9 @@ _Py_atomic_load_ssize_acquire(const Py_ssize_t *obj)
11621162
static inline void *
11631163
_Py_atomic_memcpy_ptr_store_relaxed(void *dest, void *src, size_t n)
11641164
{
1165-
// assert(((uintptr_t)dest & (sizeof (void *) - 1)) == 0);
1166-
// assert(((uintptr_t)src & (sizeof (void *) - 1)) == 0);
1167-
assert(n % (size_t)sizeof(void *) == 0);
1165+
assert(((uintptr_t)dest & (uintptr_t)(sizeof (void *) - 1)) == 0);
1166+
assert(((uintptr_t)src & (uintptr_t)(sizeof (void *) - 1)) == 0);
1167+
// assert(n % sizeof(void *) == 0);
11681168

11691169
// memcpy(dest, src, n);
11701170

@@ -1185,9 +1185,9 @@ _Py_atomic_memcpy_ptr_store_relaxed(void *dest, void *src, size_t n)
11851185
static inline void *
11861186
_Py_atomic_memmove_ptr_store_relaxed(void *dest, void *src, size_t n)
11871187
{
1188-
// assert(((uintptr_t)dest & (sizeof (void *) - 1)) == 0);
1189-
// assert(((uintptr_t)src & (sizeof (void *) - 1)) == 0);
1190-
assert(n % (size_t)sizeof(void *) == 0);
1188+
assert(((uintptr_t)dest & (uintptr_t)(sizeof (void *) - 1)) == 0);
1189+
assert(((uintptr_t)src & (uintptr_t)(sizeof (void *) - 1)) == 0);
1190+
// assert(n % sizeof(void *) == 0);
11911191

11921192
// memmove(dest, src, n);
11931193

Include/cpython/pyatomic_std.h

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1096,9 +1096,9 @@ static inline void *
10961096
_Py_atomic_memcpy_ptr_store_relaxed(void *dest, void *src, size_t n)
10971097
{
10981098
_Py_USING_STD;
1099-
// assert(((uintptr_t)dest & (sizeof (void *) - 1)) == 0);
1100-
// assert(((uintptr_t)src & (sizeof (void *) - 1)) == 0);
1101-
assert(n % (size_t)sizeof(void *) == 0);
1099+
assert(((uintptr_t)dest & (uintptr_t)(sizeof (void *) - 1)) == 0);
1100+
assert(((uintptr_t)src & (uintptr_t)(sizeof (void *) - 1)) == 0);
1101+
// assert(n % sizeof(void *) == 0);
11021102

11031103
// memcpy(dest, src, n);
11041104

@@ -1121,9 +1121,9 @@ static inline void *
11211121
_Py_atomic_memmove_ptr_store_relaxed(void *dest, void *src, size_t n)
11221122
{
11231123
_Py_USING_STD;
1124-
// assert(((uintptr_t)dest & (sizeof (void *) - 1)) == 0);
1125-
// assert(((uintptr_t)src & (sizeof (void *) - 1)) == 0);
1126-
assert(n % (size_t)sizeof(void *) == 0);
1124+
assert(((uintptr_t)dest & (uintptr_t)(sizeof (void *) - 1)) == 0);
1125+
assert(((uintptr_t)src & (uintptr_t)(sizeof (void *) - 1)) == 0);
1126+
// assert(n % sizeof(void *) == 0);
11271127

11281128
// memmove(dest, src, n);
11291129

0 commit comments

Comments
 (0)