Page MenuHomePhabricator

Update test suite to be able to run "ninja check" without X86 builtin
ClosedPublic

Authored by mehdi_amini on Aug 3 2015, 1:02 PM.

Details

Reviewers
mehdi_amini
Summary

Requires "native" feature in most places that were failing.

Diff Detail

Event Timeline

mehdi_amini updated this revision to Diff 31256.Aug 3 2015, 1:02 PM
mehdi_amini retitled this revision from to Update test suite to be able to run "ninja check" without X86 builtin.
mehdi_amini updated this object.
mehdi_amini added a subscriber: llvm-commits.

Right now it seems that this patch is doing two things - moves the target neutral MIR tests and modifies the REQUIRES lines in other tests. Wouldn't it be better if the MIR tests are moved to MIR/Generic in a separate commit?

I'll push it in two commits if you'd like, but I think it shouldn't preclude the review of this as whole (it is still highly related).

Okay, the MIR -> MIR/Generic move LGTM, but I can't comment on the other tests.

mehdi_amini accepted this revision.Aug 4 2015, 8:25 AM
mehdi_amini added a reviewer: mehdi_amini.

landed.

This revision is now accepted and ready to land.Aug 4 2015, 8:25 AM
mehdi_amini closed this revision.Aug 4 2015, 8:25 AM