]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
Fix recent rvv/base/spill testcase failures
authorJeff Law <jeffreyalaw@gmail.com>
Tue, 22 Nov 2022 23:12:45 +0000 (18:12 -0500)
committerJeff Law <jeffreyalaw@gmail.com>
Tue, 22 Nov 2022 23:12:45 +0000 (18:12 -0500)
he core issue is we're expecting the frame to have a constant size, but it
doesn't.  So when using the to_constant method we abort.

The safest thing to do is to set no shrink-wrapping components when the
frame size is not fixed.  We might be able to do better later -- iff we
know the offset to the GPRs/FPRs is fixed and fits into the appropriate
number of bits.

Bootstrapped and regression tested (C-only) on riscv64-linux-gnu.  As
expected, it fixes a bucketload of failures in rvv/base/spill-*.c.

gcc/
* config/riscv/riscv.cc (riscv_get_separate_components): Do not
do shrink-wrapping for a frame with a variable size.

gcc/config/riscv/riscv.cc

index 7ec4ce97e6cefb54dbaf43f14dbc59d75556d3ae..7bfc0e9f595d7ee38c0afe1ffe36f387b05d5ab4 100644 (file)
@@ -5340,7 +5340,8 @@ riscv_get_separate_components (void)
   bitmap_clear (components);
 
   if (riscv_use_save_libcall (&cfun->machine->frame)
-      || cfun->machine->interrupt_handler_p)
+      || cfun->machine->interrupt_handler_p
+      || !cfun->machine->frame.gp_sp_offset.is_constant ())
     return components;
 
   offset = cfun->machine->frame.gp_sp_offset.to_constant ();