From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 1666) id 106F83858D3C; Fri, 6 Oct 2023 07:07:38 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 106F83858D3C DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1696576058; bh=z9l+5i8G7RAWAKN5MEGq6j3I/wJ+XfNIX1yN493EmkI=; h=From:To:Subject:Date:From; b=SO6hvyERZ54nFRwciyXrc1hdv08NMc6ybFkJC4f2tWUs18qsv/VrpuL5dXkK/9Abq IpMFlOCVu4KldfOpGyGUcf18scZTDA/Nif7a26tLo8pUnKcezAZ4FvfStHK3jllFoJ DOF7zSPO5mfAxuz+fCdovlC5i/TgSpt1a9zTmMwc= 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: d4d841cb500924177f261c987993b3968fce9391 X-Git-Newrev: d281c06dff379be55f32a77ca112d8f4994197d5 Message-Id: <20231006070738.106F83858D3C@sourceware.org> Date: Fri, 6 Oct 2023 07:07:38 +0000 (GMT) List-Id: https://gcc.gnu.org/g:d281c06dff379be55f32a77ca112d8f4994197d5 commit d281c06dff379be55f32a77ca112d8f4994197d5 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 a4a912474a1..f29ee9f19bf 100644 --- a/gcc/tree-vect-stmts.cc +++ b/gcc/tree-vect-stmts.cc @@ -9628,6 +9628,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),