This is an archive of the discontinued LLVM Phabricator instance.

[yaml2obj] Enable and fix tests
ClosedPublic

Authored by sbc100 on Mar 28 2017, 1:36 PM.

Details

Summary

These tests were not being run because the yaml extension
wasn't be picked up by lit.

This change also fixes the tests which themselves were broken.

Diff Detail

Repository
rL LLVM

Event Timeline

sbc100 created this revision.Mar 28 2017, 1:36 PM
sbc100 retitled this revision from Enable yaml2obj tests to [yaml2obj] Enable and fix tests.Mar 28 2017, 1:36 PM
sbc100 added a reviewer: beanz.
beanz accepted this revision.Mar 28 2017, 3:18 PM

LGTM!

Thanks for catching this. I didn't even notice that I had written such bad tests!

This revision is now accepted and ready to land.Mar 28 2017, 3:18 PM
sbc100 edited the summary of this revision. (Show Details)Mar 28 2017, 4:00 PM

Would you mind committing this (I'm not yet a committer).

This revision was automatically updated to reflect the committed changes.