I'm not sure how to verify that adding the parameter won't destroy the test. I've tried to repoduce the ICE on old Arm builds of arm-none-eabi, but none of them ICE. I suppose it should be safe to add the parameter as the PR talks about the literal pools.
Ok for trunk and releases/gcc-14? -- Without this change, build fails with: .../thumb2-slow-flash-data-3.c: In function 'fn3': .../thumb2-slow-flash-data-3.c:23:3: error: too many arguments to function 'fn1' .../thumb2-slow-flash-data-3.c:10:6: note: declared here gcc/testsuite/ChangeLog: * gcc.target/arm/thumb2-slow-flash-data-3.c: Added argument to fn1 to avoid compile error. Signed-off-by: Torbjörn SVENSSON <torbjorn.svens...@foss.st.com> --- gcc/testsuite/gcc.target/arm/thumb2-slow-flash-data-3.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/testsuite/gcc.target/arm/thumb2-slow-flash-data-3.c b/gcc/testsuite/gcc.target/arm/thumb2-slow-flash-data-3.c index 09d25d62002..eb841367c91 100644 --- a/gcc/testsuite/gcc.target/arm/thumb2-slow-flash-data-3.c +++ b/gcc/testsuite/gcc.target/arm/thumb2-slow-flash-data-3.c @@ -7,7 +7,7 @@ /* From PR71607 */ float b; -void fn1 (); +void fn1 (void*); float fn2 () -- 2.25.1