From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id 5DE62388701C; Mon, 13 Apr 2020 20:44:08 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 5DE62388701C DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1586810648; bh=R+bY/xQ4n7pqwc/kpgUSPN1RNqF3q/6JFuwWur3hX/U=; h=From:To:Subject:Date:In-Reply-To:References:From; b=tBagCSmGjKXH1gv1szFw5InCmyOLBVLj0pY2atX62J3kgKcSz8lEEbBLTT1/5+jxd RRUTKqVwPdlmAq5tg9sVA5gNmuS0e9M3yZ3VKg/ngKYCeMslEbzRUYDcuqmGLtkk++ gKANXrP9ub76NMlKUVWHxf/3StM9WKRYnvdWQFnQ= From: "meissner at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug target/94557] [9 regression] r9-8486 causes several builtin instruction test case execution failures on power 9 Date: Mon, 13 Apr 2020 20:44:08 +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: 9.3.1 X-Bugzilla-Keywords: X-Bugzilla-Severity: normal X-Bugzilla-Who: meissner at gcc dot gnu.org X-Bugzilla-Status: ASSIGNED X-Bugzilla-Resolution: X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: bug_status everconfirmed cf_reconfirmed_on 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 X-BeenThere: gcc-bugs@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-bugs mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 13 Apr 2020 20:44:08 -0000 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D94557 Michael Meissner changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |ASSIGNED Ever confirmed|0 |1 Last reconfirmed| |2020-04-13 --- Comment #2 from Michael Meissner --- The issue is that with the backport patch for PR target/93932, GCC is more likely to optimize variable extracts from a vector that is in memory to be a simple load, instead of loading the vector into a vector register, and doin= g a vector extract on power9. The test cases rely on having indexes outside of the range of valid indexes= .=20 If the vector was loaded into a register, we would automatically mask the i= ndex as part of the extract. However, if we converted the operation to a single load, we did not do the masking, and the load would load some random value outside of the vector boundary. The trunk had previously had other changes that did this masking as part of= the changes for -mcpu=3Dfuture and PC-relative support. The proposed patch just makes sure the index is properly masked before it is used.=