From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm1-x332.google.com (mail-wm1-x332.google.com [IPv6:2a00:1450:4864:20::332]) by sourceware.org (Postfix) with ESMTPS id B06003858403 for ; Sun, 7 Nov 2021 21:24:25 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org B06003858403 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=embecosm.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=embecosm.com Received: by mail-wm1-x332.google.com with SMTP id d72-20020a1c1d4b000000b00331140f3dc8so10322976wmd.1 for ; Sun, 07 Nov 2021 13:24:25 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=embecosm.com; s=google; h=date:from:to:cc:subject:in-reply-to:message-id:references :user-agent:mime-version; bh=/GAmdgnMuLjQ5WqHshM17NJZJhHs/nxdUf1rwne0bUs=; b=ehCC8PCSNuWdC3bHnEJtgXxavbZUEA3OBzFXLYUYwZ2XJqPgw7k1CY/JSZt5EZAkEA d5rIehmcqqQc6P9oq1/YgGMjoiWJOjby2qPbuiKRPIjWgoLR9N+rUkR1k8G7CnpNA/Ke Dmqv6d9ydUeex+BB5hF9Hz6CQjGW+qP9R5JKH5UG9Qn+Az5pLgkd9s+nOf7NBxu4zbjG Y/pAvZLc3xUrFks5Z2wNW3IDR4QRAWoJpTW4yvNqxYfHPleGvmazfpwIXAe0HpTdUaUB X7/kBrDJlTKrjF5ihDdsXnt1lcn+YOzTGqHRqCNSEYUp7Y3kmcRCyrjsmInUhDWktE2w naEw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:from:to:cc:subject:in-reply-to:message-id :references:user-agent:mime-version; bh=/GAmdgnMuLjQ5WqHshM17NJZJhHs/nxdUf1rwne0bUs=; b=ho1KbnxeA9PoRdI9KCos1cxYvySwSK3QHCAAqBPjY9wgmwty75V+fH6fhH3cy1OReb Av/4SewDAD6y+Kpa+S2NKHjd7GkI3KwtaKqQScEuIkn6HwD4u50LX6mophU9hs43Mlcw BKNN4G9daUBFHIibPEviTGNI6gws9Hugl4A8Ac4ybqrYWfagisNm6pMRkkZn0NZ2M4lO 17ck1m7WfgumFEbm9t6iuFwFTUxdRvHafQgXDTCuDQDwxWI2jjEiGe4V77C9yuhRh69e 8O6EGPQC0tH/VfMeB0l3L+jkIS/VMaY+Kc7CFtMA3ggBwkaqqIIRUW3Rr2ic3/cgAAbN 440Q== X-Gm-Message-State: AOAM530Yhhlxus1mVdZLvD7Ou9AtGpYfy74zDSDBo3r7s5o8sROA4TvM AyYbB6jAKwkvOnW7zYYP1gTKypOILoJfmA== X-Google-Smtp-Source: ABdhPJwY5lSryE6qXVBs2JCNlKh/N3vICmqe+f1RSKOZB9i3ydZR4jy49z+taz1MFQU2OP3pKUxDUg== X-Received: by 2002:a05:600c:4153:: with SMTP id h19mr48242322wmm.142.1636320264850; Sun, 07 Nov 2021 13:24:24 -0800 (PST) Received: from tpp.orcam.me.uk (tpp.orcam.me.uk. [2001:8b0:154:0:ea6a:64ff:fe24:f2fc]) by smtp.gmail.com with ESMTPSA id 9sm18678952wry.0.2021.11.07.13.24.24 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Sun, 07 Nov 2021 13:24:24 -0800 (PST) Date: Sun, 7 Nov 2021 21:24:23 +0000 (GMT) From: "Maciej W. Rozycki" To: gcc-patches@gcc.gnu.org cc: Ulrich Weigand Subject: Re: [PATCH] PR middle-end/103059: reload: Also accept ASHIFT with indexed addressing In-Reply-To: Message-ID: References: User-Agent: Alpine 2.20 (DEB 67 2015-01-07) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII X-Spam-Status: No, score=-3.3 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, RCVD_IN_DNSWL_NONE, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org X-BeenThere: gcc-patches@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 07 Nov 2021 21:24:27 -0000 On Wed, 3 Nov 2021, Maciej W. Rozycki wrote: > I have not yet tracked down which change after commit c605a8bf9270 made > regressions appear in the test suites, however clearly the commit wasn't > as complete a change as it should have been. I'll see if I can find it > and will mention it in the final commit description if there is anything > useful in that. So regrettably it's my original commit c605a8bf9270 that caused these regressions. I don't know how it happened that it slipped through in testing; I may have had the change left in the tree that enabled LRA and did not notice it. Sorry about that, umm. > As noted in the commit description this has been regression-tested with > commit 4a960d548b7d^. I'm running regression-testing with GCC 11 right > now as well and expect results by the end of week. For the record the change does fix numerous regressions in GCC 11 too, e.g.: === gcc Summary === # of expected passes 122244 # of unexpected failures 1852 # of expected passes 122428 # of unexpected failures 1484 # of unexpected successes 5 # of expected failures 721 # of unresolved testcases 10 So either this change, if deemed safe enough, will have to go into GCC 11, or, if we're going to keep old reload yet for GCC 12, then the best course of action for GCC 11 might be reverting commit c605a8bf9270, although at the cost of regressing code quality a bit where it does build. Suggestions? Maciej