This is an archive of the discontinued LLVM Phabricator instance.

[clang] Limit clang test to ARM and AArch64 only
ClosedPublic

Authored by mgrang on Oct 20 2016, 1:21 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

mgrang updated this revision to Diff 75344.Oct 20 2016, 1:21 PM
mgrang retitled this revision from to [clang] Limit clang test to ARM only.
mgrang updated this object.
mgrang added reviewers: abdulras, honggyu.kim.
mgrang added a project: Restricted Project.
mgrang added a subscriber: cfe-commits.
rengolin added inline comments.Oct 20 2016, 1:30 PM
test/Frontend/gnu-mcount.c
1 ↗(On Diff #75344)

If you have ARM but not AArch64, this test will also fail. Can you use AND on REQUIRES?

mgrang updated this revision to Diff 75368.Oct 20 2016, 4:50 PM
mgrang retitled this revision from [clang] Limit clang test to ARM only to [clang] Limit clang test to ARM and AArch64 only.
mgrang updated this object.

Test limited only to ARM and AArch64.

mgrang marked an inline comment as done.Oct 20 2016, 4:51 PM
rengolin accepted this revision.Oct 22 2016, 1:08 PM
rengolin added a reviewer: rengolin.

I haven't tested with ARM and not AArch64 or vice-versa. I'm assuming you have. If so, LGTM. Thanks.

This revision is now accepted and ready to land.Oct 22 2016, 1:08 PM
This revision was automatically updated to reflect the committed changes.