[PATCH] D17068: [mips][microMIPS] Fix for "Cannot copy registers" assertion

Hrvoje Varga via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 17 00:12:50 PST 2016


hvarga added a comment.

In http://reviews.llvm.org/D17068#353457, @sdardis wrote:

> I haven't been able to hit the assertion on test/CodeGen/Mips/micromips-addiu.ll with "-march=mips64r6 -mattr=micromips" with r260891. Would you be able to provide another test case?


This is strange. My colleagues can also reproduce this assertion. What about the other one in this patch; did you reproduced it successfully? There are actually quite a number of test cases that are triggering the same assertion. But they are all failing because of two reasons; copying from 64-bit register into 32-bit one or copying from 32-bit register into 64-bit register. This is why I only added two test cases.

Anyway, I will list a few of those tests so you can try to reproduce it (files listed below are all located in the directory `test/CodeGen/Mips`):

- `micromips-andi.ll`
- `micromips-li.ll`
- `micromips-sw-lw-16.ll`
- `micromips-shift.ll`
- `micromips-sw-lw-16.ll`


http://reviews.llvm.org/D17068





More information about the llvm-commits mailing list