https://github.com/python/cpython/commit/4a33077fdb546db71b16d50cdd552529a9a1c910
commit: 4a33077fdb546db71b16d50cdd552529a9a1c910
branch: main
author: Serhiy Storchaka <[email protected]>
committer: serhiy-storchaka <[email protected]>
date: 2025-09-03T11:26:56+03:00
summary:
gh-138264: Fix gcc 14 compiler warnings (GH-138265)
files:
M Include/internal/pycore_stats.h
M Lib/test/test_generated_cases.py
M Modules/_sre/sre.c
M Python/executor_cases.c.h
M Python/generated_cases.c.h
M Tools/cases_generator/generators_common.py
diff --git a/Include/internal/pycore_stats.h b/Include/internal/pycore_stats.h
index ab649574f33dbf..24f239a2135b93 100644
--- a/Include/internal/pycore_stats.h
+++ b/Include/internal/pycore_stats.h
@@ -77,7 +77,7 @@ PyAPI_FUNC(PyObject*) _Py_GetSpecializationStats(void);
#define RARE_EVENT_INTERP_INC(interp, name) \
do { \
/* saturating add */ \
- int val = FT_ATOMIC_LOAD_UINT8_RELAXED(interp->rare_events.name); \
+ uint8_t val = FT_ATOMIC_LOAD_UINT8_RELAXED(interp->rare_events.name); \
if (val < UINT8_MAX) { \
FT_ATOMIC_STORE_UINT8(interp->rare_events.name, val + 1); \
} \
diff --git a/Lib/test/test_generated_cases.py b/Lib/test/test_generated_cases.py
index ec44a0f9ce3fb3..6e0e2eafd3281c 100644
--- a/Lib/test/test_generated_cases.py
+++ b/Lib/test/test_generated_cases.py
@@ -1572,7 +1572,7 @@ def test_instruction_size_macro(self):
frame->instr_ptr = next_instr;
next_instr += 1;
INSTRUCTION_STATS(OP);
- frame->return_offset = 1 ;
+ frame->return_offset = 1u ;
DISPATCH();
}
"""
diff --git a/Modules/_sre/sre.c b/Modules/_sre/sre.c
index 49eb52b635bfbb..fdf00e6499cb6b 100644
--- a/Modules/_sre/sre.c
+++ b/Modules/_sre/sre.c
@@ -2359,7 +2359,7 @@ match_getindex(MatchObject* self, PyObject* index)
}
// Check that i*2 cannot overflow to make static analyzers happy
- assert(i <= SRE_MAXGROUPS);
+ assert((size_t)i <= SRE_MAXGROUPS);
return i;
}
diff --git a/Python/executor_cases.c.h b/Python/executor_cases.c.h
index 3dcb2decc43737..635ca659394c32 100644
--- a/Python/executor_cases.c.h
+++ b/Python/executor_cases.c.h
@@ -1688,7 +1688,7 @@
_PyInterpreterFrame* pushed_frame = _PyFrame_PushUnchecked(tstate,
getitem, 2, frame);
pushed_frame->localsplus[0] = container;
pushed_frame->localsplus[1] = sub;
- frame->return_offset = 6 ;
+ frame->return_offset = 6u ;
new_frame = PyStackRef_Wrap(pushed_frame);
stack_pointer[-3] = new_frame;
stack_pointer += -2;
@@ -2057,8 +2057,8 @@
gen->gi_frame_state = FRAME_EXECUTING;
gen->gi_exc_state.previous_item = tstate->exc_info;
tstate->exc_info = &gen->gi_exc_state;
- assert( 2 + oparg <= UINT16_MAX);
- frame->return_offset = (uint16_t)( 2 + oparg);
+ assert( 2u + oparg <= UINT16_MAX);
+ frame->return_offset = (uint16_t)( 2u + oparg);
pushed_frame->previous = frame;
gen_frame = PyStackRef_Wrap(pushed_frame);
stack_pointer[-1] = gen_frame;
@@ -4610,7 +4610,7 @@
gen->gi_exc_state.previous_item = tstate->exc_info;
tstate->exc_info = &gen->gi_exc_state;
pushed_frame->previous = frame;
- frame->return_offset = (uint16_t)( 2 + oparg);
+ frame->return_offset = (uint16_t)( 2u + oparg);
gen_frame = PyStackRef_Wrap(pushed_frame);
stack_pointer[0] = gen_frame;
stack_pointer += 1;
diff --git a/Python/generated_cases.c.h b/Python/generated_cases.c.h
index 7547eaad125370..c3e37f080fa6e4 100644
--- a/Python/generated_cases.c.h
+++ b/Python/generated_cases.c.h
@@ -653,7 +653,7 @@
_PyInterpreterFrame* pushed_frame =
_PyFrame_PushUnchecked(tstate, getitem, 2, frame);
pushed_frame->localsplus[0] = container;
pushed_frame->localsplus[1] = sub;
- frame->return_offset = 6 ;
+ frame->return_offset = 6u ;
new_frame = PyStackRef_Wrap(pushed_frame);
}
// _PUSH_FRAME
@@ -1587,7 +1587,7 @@
if (new_frame == NULL) {
JUMP_TO_LABEL(error);
}
- frame->return_offset = 4 ;
+ frame->return_offset = 4u ;
DISPATCH_INLINED(new_frame);
}
STACKREFS_TO_PYOBJECTS(arguments, total_args, args_o);
@@ -2603,7 +2603,7 @@
if (new_frame == NULL) {
JUMP_TO_LABEL(error);
}
- assert( 1 == 1);
+ assert( 1u == 1);
frame->return_offset = 1;
DISPATCH_INLINED(new_frame);
}
@@ -2883,8 +2883,8 @@
if (new_frame == NULL) {
JUMP_TO_LABEL(error);
}
- assert( 4 == 1 + INLINE_CACHE_ENTRIES_CALL_KW);
- frame->return_offset = 4 ;
+ assert( 4u == 1 + INLINE_CACHE_ENTRIES_CALL_KW);
+ frame->return_offset = 4u ;
DISPATCH_INLINED(new_frame);
}
STACKREFS_TO_PYOBJECTS(arguments, total_args, args_o);
@@ -5716,7 +5716,7 @@
gen->gi_exc_state.previous_item = tstate->exc_info;
tstate->exc_info = &gen->gi_exc_state;
pushed_frame->previous = frame;
- frame->return_offset = (uint16_t)( 2 + oparg);
+ frame->return_offset = (uint16_t)( 2u + oparg);
gen_frame = PyStackRef_Wrap(pushed_frame);
}
// _PUSH_FRAME
@@ -6334,7 +6334,7 @@
if (new_frame == NULL) {
JUMP_TO_LABEL(error);
}
- frame->return_offset = 4 ;
+ frame->return_offset = 4u ;
DISPATCH_INLINED(new_frame);
}
STACKREFS_TO_PYOBJECTS(arguments, total_args, args_o);
@@ -6547,7 +6547,7 @@
if (new_frame == NULL) {
JUMP_TO_LABEL(error);
}
- assert( 1 == 1);
+ assert( 1u == 1);
frame->return_offset = 1;
DISPATCH_INLINED(new_frame);
}
@@ -6690,8 +6690,8 @@
if (new_frame == NULL) {
JUMP_TO_LABEL(error);
}
- assert( 4 == 1 + INLINE_CACHE_ENTRIES_CALL_KW);
- frame->return_offset = 4 ;
+ assert( 4u == 1 + INLINE_CACHE_ENTRIES_CALL_KW);
+ frame->return_offset = 4u ;
DISPATCH_INLINED(new_frame);
}
STACKREFS_TO_PYOBJECTS(arguments, total_args, args_o);
@@ -8060,7 +8060,7 @@
stack_pointer += -1;
assert(WITHIN_STACK_BOUNDS());
new_frame->localsplus[1] = PyStackRef_FromPyObjectNew(name);
- frame->return_offset = 10 ;
+ frame->return_offset = 10u ;
DISPATCH_INLINED(new_frame);
}
@@ -10459,8 +10459,8 @@
gen->gi_frame_state = FRAME_EXECUTING;
gen->gi_exc_state.previous_item = tstate->exc_info;
tstate->exc_info = &gen->gi_exc_state;
- assert( 2 + oparg <= UINT16_MAX);
- frame->return_offset = (uint16_t)( 2 + oparg);
+ assert( 2u + oparg <= UINT16_MAX);
+ frame->return_offset = (uint16_t)( 2u + oparg);
assert(gen_frame->previous == NULL);
gen_frame->previous = frame;
DISPATCH_INLINED(gen_frame);
@@ -10560,8 +10560,8 @@
gen->gi_frame_state = FRAME_EXECUTING;
gen->gi_exc_state.previous_item = tstate->exc_info;
tstate->exc_info = &gen->gi_exc_state;
- assert( 2 + oparg <= UINT16_MAX);
- frame->return_offset = (uint16_t)( 2 + oparg);
+ assert( 2u + oparg <= UINT16_MAX);
+ frame->return_offset = (uint16_t)( 2u + oparg);
pushed_frame->previous = frame;
gen_frame = PyStackRef_Wrap(pushed_frame);
}
diff --git a/Tools/cases_generator/generators_common.py
b/Tools/cases_generator/generators_common.py
index cc0edcdf6006fd..61e855eb003706 100644
--- a/Tools/cases_generator/generators_common.py
+++ b/Tools/cases_generator/generators_common.py
@@ -470,7 +470,7 @@ def instruction_size(self,
"""Replace the INSTRUCTION_SIZE macro with the size of the current
instruction."""
if uop.instruction_size is None:
raise analysis_error("The INSTRUCTION_SIZE macro requires
uop.instruction_size to be set", tkn)
- self.out.emit(f" {uop.instruction_size} ")
+ self.out.emit(f" {uop.instruction_size}u ")
return True
def _print_storage(self, reason:str, storage: Storage) -> None:
_______________________________________________
Python-checkins mailing list -- [email protected]
To unsubscribe send an email to [email protected]
https://mail.python.org/mailman3//lists/python-checkins.python.org
Member address: [email protected]
