This is an archive of the discontinued LLVM Phabricator instance.

[mips][mips64r6] Add align and dalign
ClosedPublic

Authored by dsanders on May 12 2014, 9:19 AM.

Diff Detail

Event Timeline

dsanders updated this revision to Diff 9316.May 12 2014, 9:19 AM
dsanders retitled this revision from to [mips][mips64r6] Add align and dalign.
dsanders updated this object.
dsanders edited the test plan for this revision. (Show Details)
jkolek accepted this revision.May 14 2014, 8:23 AM
jkolek edited edge metadata.

LGTM with a nit.

lib/Target/Mips/Mips32r6InstrInfo.td
92

For DALIGN you should have 'uimm3:$bp', and not 'uimm2:$bp' for the both DALING and ALIGN. Probably it would be best to introduce new 'Operand' parameter to class ALIGN_DESC_BASE, and pass uimm2 and uimm3 as parameters.

This revision is now accepted and ready to land.May 14 2014, 8:23 AM
dsanders closed this revision.May 15 2014, 5:13 AM