This is an archive of the discontinued LLVM Phabricator instance.

Sorted *_ENC, *_DESC classes and tests
ClosedPublic

Authored by jkolek on May 16 2014, 11:06 AM.

Details

Summary

This patch sorts *_ENC, *_DESC classes and tests. Some items are not sorted to avoid splitting of instructions from a same group.

Diff Detail

Repository
rL LLVM

Event Timeline

jkolek updated this revision to Diff 9489.May 16 2014, 11:06 AM
jkolek retitled this revision from to Sorted *_ENC, *_DESC classes and tests.
jkolek updated this object.
jkolek edited the test plan for this revision. (Show Details)
jkolek added reviewers: vmedic, dsanders.
jkolek added a subscriber: zoran.jovanovic.
dsanders accepted this revision.May 19 2014, 2:08 AM
dsanders edited edge metadata.

LGTM.

test/MC/Mips/mips64r6/valid.s
107–109 ↗(On Diff #9489)

The context is missing so I can't check in the web interface but aren't some of the branches at the end of the file at the moment? If so, they should be sorted too.

This revision is now accepted and ready to land.May 19 2014, 2:08 AM
jkolek updated this revision to Diff 9527.May 19 2014, 2:52 AM
jkolek edited edge metadata.

Here is the full context. Branches are above class.fmt instructions, they are sorted as well.

vmedic accepted this revision.May 19 2014, 6:26 AM
vmedic edited edge metadata.

LGTM

jkolek closed this revision.May 19 2014, 8:05 AM
jkolek updated this revision to Diff 9575.

Closed by commit rL209129 (authored by zjovanovic).

jkolek edited edge metadata.Nov 18 2014, 5:52 AM
jkolek added a subscriber: Unknown Object (MLST).