Retested, made independent of the rest of the series and submitted as https://gcc.gnu.org/pipermail/gcc-patches/2023-November/637662.html.
Manolis On Mon, Nov 13, 2023 at 2:43 PM Manolis Tsamis <manolis.tsa...@vrull.eu> wrote: > > Yes, my finding back then was that this is leftover code from the > initial implementation, nothing to do with the rest of the changes. > I will first re-evaluate this and test it separately from the other > series. If all is good I'll let you know so we can proceed. > > Manolis > > On Sat, Nov 11, 2023 at 12:03 AM Jeff Law <jeffreya...@gmail.com> wrote: > > > > > > > > On 8/30/23 04:14, Manolis Tsamis wrote: > > > This code used to handle register replacement issues with SUBREG before > > > simplify_replace_rtx was introduced. This should not be needed anymore as > > > new_val has the correct mode and that should be preserved by > > > simplify_replace_rtx. > > > > > > gcc/ChangeLog: > > > > > > * ifcvt.cc (noce_convert_multiple_sets_1): Remove old code. > > So is it the case that this code is supposed to no longer be needed as a > > result of your kit or it is unnecessary independent of patches 1..3? If > > the latter then it's OK for the trunk now. > > > > Jeff
