[PATCH] D16925: [mips] Support LA expansion in PIC mode

Daniel Sanders via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 5 07:45:26 PST 2016


dsanders requested changes to this revision.
dsanders added a comment.
This revision now requires changes to proceed.

The code change looks right to me but I have some comments about the test case.

Rather than make a new test file, could you extend test/MC/Mips/macro-la.s? This is mostly to keep our test directories tidy but it will also allow us to test that the other cases (e.g. 'la $5, 1') are unaffected and we'll have better coverage of the variety of expressions (e.g. 'la $5, symbol+8($6)', 'la $6, symbol+8($6)', and 'la $5, 1f').


http://reviews.llvm.org/D16925





More information about the llvm-commits mailing list