This is an archive of the discontinued LLVM Phabricator instance.

[mips][FastISel] Simplify callabi.ll by using multiple check prefixes.
ClosedPublic

Authored by vkalintiris on May 9 2015, 6:47 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

vkalintiris updated this revision to Diff 25405.May 9 2015, 6:47 AM
vkalintiris retitled this revision from to [mips][FastISel] Simplify callabi.ll by using multiple check prefixes..
vkalintiris updated this object.
vkalintiris edited the test plan for this revision. (Show Details)
vkalintiris added a reviewer: dsanders.
vkalintiris added a subscriber: Unknown Object (MLST).
dsanders accepted this revision.May 12 2015, 3:37 AM
dsanders edited edge metadata.

LGTM with a couple nits

test/CodeGen/Mips/Fast-ISel/callabi.ll
11 ↗(On Diff #25405)

Nit: Indentation
Likewise for the other ALL-LABEL's

356–363 ↗(On Diff #25405)

I know this isn't from your patch but: Please delete this and add it when it passes.

This revision is now accepted and ready to land.May 12 2015, 3:37 AM
This revision was automatically updated to reflect the committed changes.