From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 1666) id 9C4D23856974; Thu, 9 Nov 2023 13:03:33 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 9C4D23856974 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1699535013; bh=o5JtC/rJs2da5P5zPW/INia3MhayHgdsy3ySZ0te5Hc=; h=From:To:Subject:Date:From; b=hEMKlBvNBQ0IuepH4P8y8EOkiU1dJDIoAdz3qeFI0BHiSDe8l6gxc8nI+4hgK2iBX t8uiRWya2KscThAMIIxOjQXfIULRB1XBKBPk5hW2o2xpDxoOOZ5v8oIQYE9Aa1wjaE nJTfTjhtmtb/ahpypzCwakFYSoz4jBWkVBIKqnvA= 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: fdbab83136579a47e7703e36850714e19c291939 X-Git-Newrev: 935d0eeaeb2054e6ccf00355dff89c5ab69ebd40 Message-Id: <20231109130333.9C4D23856974@sourceware.org> Date: Thu, 9 Nov 2023 13:03:33 +0000 (GMT) List-Id: https://gcc.gnu.org/g:935d0eeaeb2054e6ccf00355dff89c5ab69ebd40 commit 935d0eeaeb2054e6ccf00355dff89c5ab69ebd40 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 d383722e968..ddbcb0c50dc 100644 --- a/gcc/tree-vect-stmts.cc +++ b/gcc/tree-vect-stmts.cc @@ -9870,6 +9870,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),