Hi All, This patch relaxes the subreg invariant that you can only change modes or make it paradoxical in one conversion. i.e. it now allows subreg:V2DI (reg:DF ..))
This is well defined in the generic sense and allowing it would enable you to write RTL without the extra moves which can be interfered with by combine. Patch has been pre-approved[1], but giving people chance to object [1] https://gcc.gnu.org/pipermail/gcc-patches/2023-September/629119.html Bootstrapped Regtested on aarch64-none-linux-gnu, x86_64-linux-gnu, powerpc64le-unknown-linux-gnu and no issues. Pushed to master. Thanks, Tamar gcc/ChangeLog: * emit-rtl.cc (validate_subreg): Relax subreg rule. --- inline copy of patch -- diff --git a/gcc/emit-rtl.cc b/gcc/emit-rtl.cc index f6276a2d0b627fd5b2a742795a55423f449e9d93..ea85a46a0f014b81aeb76d69d19a57233dd70bdc 100644 --- a/gcc/emit-rtl.cc +++ b/gcc/emit-rtl.cc @@ -947,7 +947,7 @@ validate_subreg (machine_mode omode, machine_mode imode, in post-reload splitters that make arbitrarily mode changes to the registers themselves. */ else if (VECTOR_MODE_P (omode) - && GET_MODE_INNER (omode) == GET_MODE_INNER (imode)) + && GET_MODE_UNIT_SIZE (omode) == GET_MODE_UNIT_SIZE (imode)) ; /* Subregs involving floating point modes are not allowed to change size unless it's an insert into a complex mode. --
diff --git a/gcc/emit-rtl.cc b/gcc/emit-rtl.cc index f6276a2d0b627fd5b2a742795a55423f449e9d93..ea85a46a0f014b81aeb76d69d19a57233dd70bdc 100644 --- a/gcc/emit-rtl.cc +++ b/gcc/emit-rtl.cc @@ -947,7 +947,7 @@ validate_subreg (machine_mode omode, machine_mode imode, in post-reload splitters that make arbitrarily mode changes to the registers themselves. */ else if (VECTOR_MODE_P (omode) - && GET_MODE_INNER (omode) == GET_MODE_INNER (imode)) + && GET_MODE_UNIT_SIZE (omode) == GET_MODE_UNIT_SIZE (imode)) ; /* Subregs involving floating point modes are not allowed to change size unless it's an insert into a complex mode.