This is the mail archive of the gcc-bugs@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[Bug target/52908] xop-mul-1:f9 miscompiled on bulldozer (-mxop)


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

--- Comment #6 from Venkataramanan <venkataramanan.kumar at amd dot com> 2012-05-09 03:13:01 UTC ---
(In reply to comment #5)
> (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.

Hi Uros, 

I did a complete boot strap test and Ok with the patch. We will adjust the test
case xop-imul32widen-vector.c to scan for vmuldq.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]