[llvm] r197466 - Add -mcpu=z10 to SystemZ tests.
Richard Sandiford
rsandifo at linux.vnet.ibm.com
Wed Dec 18 03:32:36 PST 2013
Hi Andrew,
Andrew Trick <atrick at apple.com> writes:
> URL: http://llvm.org/viewvc/llvm-project?rev=197466&view=rev
> Log:
> Add -mcpu=z10 to SystemZ tests.
>
> Modified:
> llvm/trunk/test/CodeGen/SystemZ/int-div-01.ll
> llvm/trunk/test/CodeGen/SystemZ/int-div-03.ll
> llvm/trunk/test/CodeGen/SystemZ/int-div-04.ll
> llvm/trunk/test/CodeGen/SystemZ/int-mul-08.ll
I tried winding back to r197465 and it looks like you added these because
on z196 you got:
ogrk %r2, %r2, %r3 instead of ogr %r2, %r3
and ork %r2, %r2, %r3 instead of or %r2, %r3
Is that right? We don't really want to use the three-address forms
(OGRK and ORK) when the two-address form would do. In these cases
I think the destination and first source are natural ties so we really
should be using OGR and OR for z196 too.
It looks like current trunk (r197599) works again for -mcpu=z196.
Would it be OK to revert this for now?
Thanks,
Richard
More information about the llvm-commits
mailing list