From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gcc-bugs-return-390805-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org>
Received: (qmail 24930 invoked by alias); 4 May 2012 12:52:08 -0000
Received: (qmail 24912 invoked by uid 22791); 4 May 2012 12:52:07 -0000
X-SWARE-Spam-Status: No, hits=-4.2 required=5.0	tests=ALL_TRUSTED,AWL,BAYES_00,KHOP_THREADED,TW_SD,TW_VM,TW_VP,TW_ZJ
X-Spam-Check-By: sourceware.org
Received: from localhost (HELO gcc.gnu.org) (127.0.0.1)    by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Fri, 04 May 2012 12:51:54 +0000
From: "ubizjak at gmail dot com" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug target/52908] xop-mul-1:f9 miscompiled on bulldozer (-mxop)
Date: Fri, 04 May 2012 12:52:00 -0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: target
X-Bugzilla-Keywords:
X-Bugzilla-Severity: normal
X-Bugzilla-Who: ubizjak at gmail dot com
X-Bugzilla-Status: ASSIGNED
X-Bugzilla-Priority: P3
X-Bugzilla-Assigned-To: ubizjak at gmail dot com
X-Bugzilla-Target-Milestone: ---
X-Bugzilla-Changed-Fields:
Message-ID: <bug-52908-4-8JZVsnJbJv@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-52908-4@http.gcc.gnu.org/bugzilla/>
References: <bug-52908-4@http.gcc.gnu.org/bugzilla/>
X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/
Auto-Submitted: auto-generated
Content-Type: text/plain; charset="UTF-8"
MIME-Version: 1.0
Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
List-Id: <gcc-bugs.gcc.gnu.org>
List-Archive: <http://gcc.gnu.org/ml/gcc-bugs/>
List-Post: <mailto:gcc-bugs@gcc.gnu.org>
List-Help: <mailto:gcc-bugs-help@gcc.gnu.org>
Sender: gcc-bugs-owner@gcc.gnu.org
X-SW-Source: 2012-05/txt/msg00423.txt.bz2

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52908

--- Comment #5 from Uros Bizjak <ubizjak at gmail dot com> 2012-05-04 12:51:49 UTC ---
(In reply to comment #4)

> A Quick make check on i386.exp result is shown below: 
> 
> Tests that now fail, but worked before:
> 
> gcc.target/i386/xop-imul32widen-vector.c scan-assembler vpmacsdql

This is expected, there is no need to emit vpmacsdql, IMO vpmuldq works as good
(if not even better, since we don't have to preload accumulator with zero). The
test should be either changed to really emit vpmacsdql, or asm scan should be
adjusted.

> Tests that now work, but didn't before:
> 
> gcc.target/i386/xop-mul-1.c execution test

Yes this one should be fixed with the patch.

If the change vmpacsdql -> vpmuldq is OK with you, I can commit the patch, but
I'd ask you for full bootstrap/regression test.