Message ID | oredfgytmv.fsf_-_@lxoliva.fsfla.org |
---|---|
State | Unresolved |
Headers |
Return-Path: <gcc-patches-bounces+ouuuleilei=gmail.com@gcc.gnu.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:7300:2483:b0:fb:cd0c:d3e with SMTP id q3csp212668dyi; Wed, 20 Dec 2023 22:09:41 -0800 (PST) X-Google-Smtp-Source: AGHT+IGzFjolRviorDkg8cZIYOvoHfNHULvbIPHnw1N8xbW8z7xJn2L7Q7rQqiiAhnJQDjVxBJVR X-Received: by 2002:ae9:e012:0:b0:781:1f81:1fbe with SMTP id m18-20020ae9e012000000b007811f811fbemr325130qkk.50.1703138981273; Wed, 20 Dec 2023 22:09:41 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1703138981; cv=pass; d=google.com; s=arc-20160816; b=x4BBzYCZcA+h6a97QnFqnM5v3/YPKN0mH1avi82yo4qQl+OOqD6SWKsaLtHkPSMIw/ 1/B8EBq5y7umoUkXJBiTS6czGZGCG5higsnzHfv+pa+chzdaQd43S1OpnwdfX7DmP5sd 1fYlCCt0mgYBl3Z0S1xmL5qVXuDbl3j48GY0h11hB0IVR5KosZCcBnS8e0rCkgxe4eB9 7qaw1oK0lAxFGmGer4e/S5hB9Oit3TYw0tFqP7deQpEvqJn9Vs7Jh5SbGG1t4iQQ7pYg 96aIUApzzDvMDayfS6vQVj9JA/nh98UNuXpgBO6SkOFWCDKgB/NWv+jp94cwRhN/tq7S Sygw== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=errors-to:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:content-transfer-encoding :mime-version:user-agent:message-id:in-reply-to:date:references :organization:subject:cc:to:from:dkim-signature:arc-filter :dmarc-filter:delivered-to; bh=vHgxeJtNECMvkNzpBkcnrQke0grf1CvQ2/+XorW+6CM=; fh=jW0Z4VeKI+ZH963cTdrDZio3DYNhvWfEMv/RN9QeeXo=; b=h0mNyI+xNetTzazFRbUVvvdcGAHNSQrsNOAIN2zlzXahxw1h1WNavvK3LjKEPZCY2W R9PH///Oscks+nIJqb2hkZHEEMs7WdVf5PKRFZYTyyVRn7IpfKYwx4GYLV78+KGJbkGb ht8AqrI6GsfdTBbqBC0Y+Y0U2YVyz198FgQNDw2vMupyLZSlgD2QfIne9Nd0XtZi3igw 8HqzUtiwmOhMGWK3mqb33gQ6mFzIIh4fI/Fy6zwBUrr4IDN9M65s3xT0xsi3EFWiHBVU KaeF62eBq1O48VsBJBqcPfd1ctZPyHlfigNm3CXVMV2lcszuzRqvQulkcrvLxTsIp320 5eHQ== ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@adacore.com header.s=google header.b=Ag4tmgDF; arc=pass (i=1); spf=pass (google.com: domain of gcc-patches-bounces+ouuuleilei=gmail.com@gcc.gnu.org designates 8.43.85.97 as permitted sender) smtp.mailfrom="gcc-patches-bounces+ouuuleilei=gmail.com@gcc.gnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=adacore.com Received: from server2.sourceware.org (server2.sourceware.org. [8.43.85.97]) by mx.google.com with ESMTPS id bp42-20020a05620a45aa00b0077f01c246aesi1589846qkb.304.2023.12.20.22.09.41 for <ouuuleilei@gmail.com> (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 20 Dec 2023 22:09:41 -0800 (PST) Received-SPF: pass (google.com: domain of gcc-patches-bounces+ouuuleilei=gmail.com@gcc.gnu.org designates 8.43.85.97 as permitted sender) client-ip=8.43.85.97; Authentication-Results: mx.google.com; dkim=pass header.i=@adacore.com header.s=google header.b=Ag4tmgDF; arc=pass (i=1); spf=pass (google.com: domain of gcc-patches-bounces+ouuuleilei=gmail.com@gcc.gnu.org designates 8.43.85.97 as permitted sender) smtp.mailfrom="gcc-patches-bounces+ouuuleilei=gmail.com@gcc.gnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=adacore.com Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 030B33861891 for <ouuuleilei@gmail.com>; Thu, 21 Dec 2023 06:09:41 +0000 (GMT) X-Original-To: gcc-patches@gcc.gnu.org Delivered-To: gcc-patches@gcc.gnu.org Received: from mail-pl1-x630.google.com (mail-pl1-x630.google.com [IPv6:2607:f8b0:4864:20::630]) by sourceware.org (Postfix) with ESMTPS id 886DF3858404 for <gcc-patches@gcc.gnu.org>; Thu, 21 Dec 2023 06:09:16 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 886DF3858404 Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=adacore.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=adacore.com ARC-Filter: OpenARC Filter v1.0.0 sourceware.org 886DF3858404 Authentication-Results: server2.sourceware.org; arc=none smtp.remote-ip=2607:f8b0:4864:20::630 ARC-Seal: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1703138958; cv=none; b=ty7PEGbHOTw1TMaMmLBFL3f6sLWUE1e2A080kZ1/wiJ018kxf/m6AK8m7AcJZGTbEtt7Zp9IJG6YqlRvmhsuHbSBgDtXd6De703BfCUv0+X0riVmscd3mJ83/2l51jwwoRFp6+Q1zMPoT9RP6AjcEvUA//cfauiENWx3dd4jHOs= ARC-Message-Signature: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1703138958; c=relaxed/simple; bh=GCqCRhTuahklm+tYv+t5h9/EzQGC0Uw77rIFli4hNb0=; h=DKIM-Signature:From:To:Subject:Date:Message-ID:MIME-Version; b=SLLwdbevHIcltVMPtr6BNs0a8oyu4MPKFTatXNbzzWyAmQa/Y1YHm7Nz29++sG4fZph0Vid6e6Soe5GseT8ZR4I24Vt4aabI3NJKwhv6nxBY1PH1qFLiBa/k3BlFj82pYnzrIkqTmNEaF/yWtJmZZEujkIVg/ybDTP8/J6c2aFw= ARC-Authentication-Results: i=1; server2.sourceware.org Received: by mail-pl1-x630.google.com with SMTP id d9443c01a7336-1d3536cd414so3885015ad.2 for <gcc-patches@gcc.gnu.org>; Wed, 20 Dec 2023 22:09:16 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=adacore.com; s=google; t=1703138954; x=1703743754; darn=gcc.gnu.org; h=content-transfer-encoding:mime-version:user-agent:message-id :in-reply-to:date:references:organization:subject:cc:to:from:from:to :cc:subject:date:message-id:reply-to; bh=vHgxeJtNECMvkNzpBkcnrQke0grf1CvQ2/+XorW+6CM=; b=Ag4tmgDF5ErCc3OgJx0JBJy6dmqphDaMrrFcw7KJzfSeZ5YEKIE539KsI42VZaYugD puw+t4uuEkD+n+aZtia+U6Ghq6tp4S2b0WYNDT62I7lUVFFpdw01Sm7WpujjjVvtz/XG wldbPtWHZ5XXuFF9hLpzwmZCsSZSWY2agQ+Y50l1+TeHmkkyQQSJbWdpD2Rq1kRXhAYc 8ScK/VSUiZ3LdXVj2RZsn+ZPjR/muywIcRMR/LxQE410JpdZfVRKBy49kVD/ievtDkt3 N02bc0q5ilWjhtFxykZckhbAmOXptSDaQSzdj5nQ8TwthQiwr+xEymKjhSllSZ0WzaLX sRgw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1703138954; x=1703743754; h=content-transfer-encoding:mime-version:user-agent:message-id :in-reply-to:date:references:organization:subject:cc:to:from :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=vHgxeJtNECMvkNzpBkcnrQke0grf1CvQ2/+XorW+6CM=; b=pzYHTEs54ug+oRYHEpv63U9otYtnjmA+wsy2BBuxQ9yaqRlOEPmqn6RKaLBG3Kr8EA uPoGnpKpiu80x3LL/8lyukSIYdNGEwDT2nyym/B7Ic3wLEZFQiUBoZ+nVaLzVqAaV2no 4gDrxaVEz6VIk7eOaCVfV2c2imxjgdjVpLevkRx/g7Zbb9SPrQRNeXNsSUCQuX99dies 8VJ/sgrzNK7W/INqkx8rENdT41ENGzjwHEH8O44djrWgBqXIEb/TzMIr9Gkj2cuGTPfe LuAVpwdXC5RSDp3adUcvS0kWZZ1j7xJXyUwtOjJwXczRnUWYQrLliILLRM+Q+j3PYRHx u1xA== X-Gm-Message-State: AOJu0YwFrzpX0KGgDJzN1Ej8MpBeR3NA3B2ldTNUhYFlG7rB3+FjTXXB gDVG2vVl4OIL1k5M0XKGdYDsiw== X-Received: by 2002:a17:902:ce90:b0:1d3:df15:e032 with SMTP id f16-20020a170902ce9000b001d3df15e032mr4105978plg.21.1703138954640; Wed, 20 Dec 2023 22:09:14 -0800 (PST) Received: from free.home ([2804:7f1:2080:e946:8ecf:6d68:6986:b9a]) by smtp.gmail.com with ESMTPSA id iw17-20020a170903045100b001d3b7c5776asm739705plb.160.2023.12.20.22.09.12 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 20 Dec 2023 22:09:13 -0800 (PST) Received: from livre (livre.home [172.31.160.2]) by free.home (8.15.2/8.15.2) with ESMTPS id 3BL68wvY735533 (version=TLSv1.3 cipher=TLS_AES_256_GCM_SHA384 bits=256 verify=NOT); Thu, 21 Dec 2023 03:09:01 -0300 From: Alexandre Oliva <oliva@adacore.com> To: Richard Biener <richard.guenther@gmail.com> Cc: gcc-patches@gcc.gnu.org Subject: [PATCH 2/2 FYI] -finline-stringops: drop obsolete comment [PR112778] Organization: Free thinker, does not speak for AdaCore References: <orplzg9jqu.fsf@lxoliva.fsfla.org> <orcyvfambh.fsf@lxoliva.fsfla.org> <CAFiYyc3Xzdeta_yP=PgjD+sSiYKoMRirCW2dwW6GkU=0f3jSNQ@mail.gmail.com> Date: Thu, 21 Dec 2023 03:08:56 -0300 In-Reply-To: <CAFiYyc3Xzdeta_yP=PgjD+sSiYKoMRirCW2dwW6GkU=0f3jSNQ@mail.gmail.com> (Richard Biener's message of "Mon, 11 Dec 2023 08:36:42 +0100") Message-ID: <oredfgytmv.fsf_-_@lxoliva.fsfla.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Scanned-By: MIMEDefang 2.84 X-Spam-Status: No, score=-12.4 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, RCVD_IN_DNSWL_NONE, SPF_HELO_NONE, SPF_PASS, TXREP, T_SCC_BODY_TEXT_LINE, WEIRD_QUOTING autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org X-BeenThere: gcc-patches@gcc.gnu.org X-Mailman-Version: 2.1.30 Precedence: list List-Id: Gcc-patches mailing list <gcc-patches.gcc.gnu.org> List-Unsubscribe: <https://gcc.gnu.org/mailman/options/gcc-patches>, <mailto:gcc-patches-request@gcc.gnu.org?subject=unsubscribe> List-Archive: <https://gcc.gnu.org/pipermail/gcc-patches/> List-Post: <mailto:gcc-patches@gcc.gnu.org> List-Help: <mailto:gcc-patches-request@gcc.gnu.org?subject=help> List-Subscribe: <https://gcc.gnu.org/mailman/listinfo/gcc-patches>, <mailto:gcc-patches-request@gcc.gnu.org?subject=subscribe> Errors-To: gcc-patches-bounces+ouuuleilei=gmail.com@gcc.gnu.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1785870660087399577 X-GMAIL-MSGID: 1785870660087399577 |
Series |
[v2] -finline-stringops: check base blksize for memset [PR112778]
|
|
Checks
Context | Check | Description |
---|---|---|
snail/gcc-patch-check | warning | Git am fail log |
Commit Message
Alexandre Oliva
Dec. 21, 2023, 6:08 a.m. UTC
On Dec 11, 2023, Richard Biener <richard.guenther@gmail.com> wrote: > On Sat, Dec 9, 2023 at 8:05 AM Alexandre Oliva <oliva@adacore.com> wrote: >> PR target/112778 >> * builtins.cc (can_store_by_multiple_pieces): New. >> (try_store_by_multiple_pieces): Call it. >> +/* Check that store_by_pieces allows BITS + LEN (so that we don't >> + expand something too unreasonably long), and every power of 2 in >> + BITS. It is assumed that LEN has already been tested by >> + itself. */ >> +static bool >> +can_store_by_multiple_pieces (unsigned HOST_WIDE_INT bits, When fixing the PR, I failed to remove the comment that raised the very concern that the PR confirmed, and that the earlier patch for the PR fixed. I'm checking this in as obvious. for gcc/ChangeLog PR target/112778 * builtins.cc (try_store_by_multiple_pieces): Drop obsolete comment. --- gcc/builtins.cc | 4 ---- 1 file changed, 4 deletions(-)
diff --git a/gcc/builtins.cc b/gcc/builtins.cc index 0f64feeedbad6..125ea158ebfad 100644 --- a/gcc/builtins.cc +++ b/gcc/builtins.cc @@ -4491,10 +4491,6 @@ try_store_by_multiple_pieces (rtx to, rtx len, unsigned int ctz_len, if (max_len >> max_bits > min_len >> max_bits) tst_bits = max_bits; } - /* ??? Do we have to check that all powers of two lengths from - max_bits down to ctz_len pass can_store_by_pieces? As in, could - it possibly be that xlenest passes while smaller power-of-two - sizes don't? */ by_pieces_constfn constfun; void *constfundata;