This is an archive of the discontinued LLVM Phabricator instance.

[mips][microMIPS] Add microMIPS32r6 and microMIPS64r6 tests for existing 16-bit LBU16, LHU16, LW16, LWGP and LWSP instructions
ClosedPublic

Authored by hvarga on Jul 6 2015, 5:04 AM.

Details

Summary

The patch adds microMIPS32r6 and microMIPS64r6 tests for existing 16-bit LBU16, LHU16, LW16, LWGP and LWSP instructions.

Diff Detail

Repository
rL LLVM

Event Timeline

hvarga updated this revision to Diff 29078.Jul 6 2015, 5:04 AM
hvarga retitled this revision from to [mips][microMIPS] Add microMIPS32r6 and microMIPS64r6 tests for existing 16-bit LBU16, LHU16, LW16, LWGP and LWSP instructions.
hvarga updated this object.
hvarga added subscribers: llvm-commits, petarj.
dsanders accepted this revision.Aug 13 2015, 6:03 AM
dsanders edited edge metadata.

LGTM with a test for the lbu special case

test/MC/Mips/micromips32r6/valid.s
65 ↗(On Diff #29078)

An offset of -1 is a special case for lbu16 (the encoding is unsigned except for 0xf which is -1). We should test that too.

test/MC/Mips/micromips64r6/valid.s
15 ↗(On Diff #29078)

Same here.

This revision is now accepted and ready to land.Aug 13 2015, 6:03 AM
hvarga updated this revision to Diff 32379.Aug 18 2015, 12:03 AM
hvarga edited edge metadata.

Updated according the comments from dsanders.

This revision was automatically updated to reflect the committed changes.