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.

Reply via email to