From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-lj1-x22b.google.com (mail-lj1-x22b.google.com [IPv6:2a00:1450:4864:20::22b]) by sourceware.org (Postfix) with ESMTPS id F22BA3858D20 for ; Tue, 15 Feb 2022 13:32:27 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org F22BA3858D20 Received: by mail-lj1-x22b.google.com with SMTP id b38so10964057ljr.12 for ; Tue, 15 Feb 2022 05:32:27 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=Vqxzdb4Defgq3uaIutOr/pq1gTx0BO4lviPnfQU1wig=; b=dYV70pU6jbR5nVMHLTnaRkbYh9t1WEC0H9lXVX4uZdY9/aEq1alc7z+XOB71KPmyKW koTP0laBiGkzgpVx3q6MrU4QToCmVIY/l5FBVOmutdNGnlp4ovZLZLGPsNa1GF6pooIC P17SHwoN7E1XxB+fItjQ/QTCzKn6kEGxLFCrt8i2Vx91Jpmu9hUcSoaYy/edi4K7x+uZ Cq3fZsO//elQqAiK34jEpNiZtDnFOGW4080L8WVxxP9Qe4GrLnJ1Rr07lHneErTcYqXc 7mHTdGqs24WHQMjgpLPCVAaKqscGWrmJKJ+fg2BtxAF6ElRsAxRh/czjp+Mpv2LYXp6y YNQg== X-Gm-Message-State: AOAM531/vqR7nKh/UUDTdtWmHWMHIBa0Cgs+5baB05iNV2IB9J2DmFA7 fDqNfFRxXny8OM2e/ImY/IWaS5wQNhKSPJGCx3wC25i4 X-Google-Smtp-Source: ABdhPJw25IWwmrnbFqCQi0bGloDXO/4T7i+4q89O5pul167MTrvNoDbi9Tg3tfEy5/kKy+9WUfWLlNhBa3LrmwX7dn8= X-Received: by 2002:a05:651c:2116:: with SMTP id a22mr2676332ljq.176.1644931946919; Tue, 15 Feb 2022 05:32:26 -0800 (PST) MIME-Version: 1.0 References: <20220109122946.2754917-1-goldstein.w.n@gmail.com> <20220110213540.1258344-1-goldstein.w.n@gmail.com> <20220110213540.1258344-5-goldstein.w.n@gmail.com> <87ilthv8s9.fsf@igel.home> <87y22dtg19.fsf@igel.home> <87tud1tf5d.fsf@igel.home> <87czjomgqn.fsf@igel.home> In-Reply-To: From: Noah Goldstein Date: Tue, 15 Feb 2022 07:32:15 -0600 Message-ID: Subject: Re: [PATCH v3 5/7] x86: Optimize strcmp-avx2.S To: Andreas Schwab Cc: "H.J. Lu" , Sunil K Pandey , Noah Goldstein via Libc-alpha Content-Type: text/plain; charset="UTF-8" X-Spam-Status: No, score=-2.4 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, FREEMAIL_FROM, RCVD_IN_DNSWL_NONE, SPF_HELO_NONE, SPF_PASS, TXREP, T_SCC_BODY_TEXT_LINE 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: libc-alpha@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Libc-alpha mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 15 Feb 2022 13:32:29 -0000 On Tue, Feb 15, 2022 at 7:09 AM Noah Goldstein wrote: > > On Tue, Feb 15, 2022 at 6:58 AM Noah Goldstein wrote: > > > > On Tue, Feb 15, 2022 at 6:55 AM Andreas Schwab wrote: > > > > > > #0 0x00007f4fd4a61df3 in __strncmp_avx2_w (a=0x55aaa1906ffa "tst-tlsmod%", > > > b=0x55aaa1940fed "tst-tls-manydynamic73mod", c=10) > > > at ../sysdeps/x86_64/multiarch/strncmp-avx2_w.c:11 > > > 11 if (n1 != n2) asm("hlt"); > > > > I'll check that input thanks! > > > > One thing, your check has some false positives as all that matters is > > n1 / n2 have > > same zero/non-zero status or same sign. > > Confirmed. Sorry for the bug, will ping back when fix is up. Found a bug (hopefully the bug) in strncmp. Did you see this at all in strcmp-avx2 or was it just the commit you were referencing? > > > > > > -- > > > Andreas Schwab, schwab@linux-m68k.org > > > GPG Key fingerprint = 7578 EB47 D4E5 4D69 2510 2552 DF73 E780 A9DA AEC1 > > > "And now for something completely different."