From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 1666) id E32103858C2B; Thu, 2 Nov 2023 13:59:50 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org E32103858C2B DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1698933590; bh=jzjnMXxDMwbiy6nG5EaXYLtnsmya7snQ6Lapdwz8uEE=; h=From:To:Subject:Date:From; b=RIsNLywIUAbw9vXY8Tn4vTOlaOSmkMnqiwp0sIKJ7avY8vHuZmKrro2h4iYkqOY1A QYHToXtXIjIU9GwZEjA4RKhVZPiyybgwfXYUcmVbKSMniPzA1ZJRpEXGgFAsYVfgL2 oqyMFPFaKHCPDhsclvTJzf+LyTQjAV0LpaqMEUW8= Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: Richard Biener To: gcc-cvs@gcc.gnu.org Subject: [gcc(refs/users/rguenth/heads/vect-force-slp)] Add FIXME note regarding gcc.dg/vect/pr60276.c runfail with single-lane SLP X-Act-Checkin: gcc X-Git-Author: Richard Biener X-Git-Refname: refs/users/rguenth/heads/vect-force-slp X-Git-Oldrev: 50f5fc2a8ceed0ebee78e290111a936ca101193b X-Git-Newrev: dd1809c6dd7574a47979b9f0a2d5a801b42e6b4f Message-Id: <20231102135950.E32103858C2B@sourceware.org> Date: Thu, 2 Nov 2023 13:59:50 +0000 (GMT) List-Id: https://gcc.gnu.org/g:dd1809c6dd7574a47979b9f0a2d5a801b42e6b4f commit dd1809c6dd7574a47979b9f0a2d5a801b42e6b4f Author: Richard Biener Date: Wed Oct 4 14:34:18 2023 +0200 Add FIXME note regarding gcc.dg/vect/pr60276.c runfail with single-lane SLP * tree-vect-stmts.cc (vectorizable_load): Add FIXME to PR60276 fix. Diff: --- gcc/tree-vect-stmts.cc | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/gcc/tree-vect-stmts.cc b/gcc/tree-vect-stmts.cc index 509e76b9f7b2..a7a824aafe88 100644 --- a/gcc/tree-vect-stmts.cc +++ b/gcc/tree-vect-stmts.cc @@ -9890,6 +9890,11 @@ vectorizable_load (vec_info *vinfo, /* Invalidate assumptions made by dependence analysis when vectorization on the unrolled body effectively re-orders stmts. */ + /* ??? This fails to trigger with single-lane SLP, gcc.dg/vect/pr60276.c, + but simply removing the ncopies > 1 conditional here (and below) will + cause FAILs of gcc.dg/vect/no-vfa-vect-depend-3.c and + gcc.dg/vect/tsvc/vect-tsvc-s3251.c. The original fix (for PR60276) + needs to be re-thought. */ if (ncopies > 1 && STMT_VINFO_MIN_NEG_DIST (stmt_info) != 0 && maybe_gt (LOOP_VINFO_VECT_FACTOR (loop_vinfo),