From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id 4EEEB3858D32; Mon, 7 Nov 2022 08:27:48 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 4EEEB3858D32 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1667809668; bh=gHKId6UIPNIWf0Oet7jSQTS3KPOUUjWHbm7giruAf6Y=; h=From:To:Subject:Date:In-Reply-To:References:From; b=BL2/JpmGnw3+ubkdBhucPe5m86dAP/8eGwRXNtKCsDZhEl20MxuUaqyYMbcmZ6LLi XUhAqOCfXxasmPDWzEl0T4xD4BLjbh/R9wA65ckgfZPmGoSS/x4W4/RA5aMxWrlN7k gh4B3l/5bxgHcP+4e0xmUswiTqDWjIVTtpLYYALY= From: "crazylht at gmail dot com" To: gcc-bugs@gcc.gnu.org Subject: [Bug target/107546] [10/11/12/13 Regression] simd, redundant pcmpeqb and pxor Date: Mon, 07 Nov 2022 08:27:47 +0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: target X-Bugzilla-Version: 12.2.0 X-Bugzilla-Keywords: missed-optimization X-Bugzilla-Severity: normal X-Bugzilla-Who: crazylht at gmail dot com X-Bugzilla-Status: NEW X-Bugzilla-Resolution: X-Bugzilla-Priority: P2 X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org X-Bugzilla-Target-Milestone: 10.5 X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 List-Id: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D107546 --- Comment #7 from Hongtao.liu --- (In reply to Hongtao.liu from comment #3) > Failed to match this instruction: > (set (reg:V16QI 95) > (eq:V16QI (gt:V16QI (subreg:V16QI (reg:V2DI 89 [ MEM[(const __m128i_u= * > {ref-all})p_2(D)] ]) 0) > (mem/u/c:V16QI (symbol_ref/u:DI ("*.LC0") [flags 0x2]) [0 S16 > A128])) > (const_vector:V16QI [ > (const_int 0 [0]) repeated x16 > ]))) >=20 >=20 > I think rtl can simplify vector comparison >=20 > (eq (gt op1 op2) const0_rtx) to just (gt op2 op1). No, it's not, it's (le op1 op2) which is not supported under sse2 ,and this= is the exact reason we do negate the result in the backend.=