[PATCH] D17079: [mips] Added -gpsize=0 to a test case as this is required in order to emit the same code as GAS.

Scott Egerton via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 3 03:21:53 PST 2016


s.egerton added a comment.

In http://reviews.llvm.org/D16455#343001, @dsanders wrote:

> This change looks correct but it appears to expose a bug. One of the tests in macro-la-64bit.s emits different code to GAS. IAS gives:
>
>   2f0:   34068000        li      a2,0x8000
>   2f4:   00c6302d        daddu   a2,a2,a2
>
> where the li clobbers the third operand of the daddu, whereas GAS emits:
>
>   2f0:   34018000        li      at,0x8000
>   2f4:   0026302d        daddu   a2,at,a2


I cannot recall the exact difference but I believe it is the 32bit equivalent of this difference. I can get more details on this if required. The correct behavior for when the -gpsize=0 option is not given has not been implemented yet. When it is implemented this test will start to fail without this option present.


http://reviews.llvm.org/D17079





More information about the llvm-commits mailing list