This required the isSuperOrSubRegisterEq function in order to avoid
depending on the equality operator when comparing registers. This was
added in http://reviews.llvm.org/rL258400
Details
Details
- Reviewers
vkalintiris dsanders
Diff Detail
Diff Detail
Event Timeline
Comment Actions
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
Could you correct the relevant CHECK lines and fix the bug?
Comment Actions
Added the -gpsize=0 option to the test cases so that the IAS will emit the same code as GAS.
Comment Actions
Just removing a 3 year old patch from my queue. Feel free to re-add me if this is still needed and you want me to take another look