This is an archive of the discontinued LLVM Phabricator instance.

Add "REQUIRES-ANY" feature test
ClosedPublic

Authored by EricWF on May 27 2016, 3:04 PM.

Details

Summary

This patch adds a "REQUIRES-ANY" feature test that is disjunctive. This marks a test as UNSUPPORTED if none of the specified features are available.

Libc++ has the need to write feature test such as // REQUIRES-ANY: c++98, c++03 when testing of behavior that is specific to older dialects but has since changed.

Diff Detail

Event Timeline

EricWF updated this revision to Diff 58844.May 27 2016, 3:04 PM
EricWF retitled this revision from to Add "REQUIRES-ANY" feature test.
EricWF updated this object.
EricWF added reviewers: ddunbar, rnk.
EricWF set the repository for this revision to rL LLVM.
EricWF added subscribers: cfe-commits, llvm-commits.
rnk accepted this revision.May 31 2016, 1:45 PM
rnk edited edge metadata.

lgtm

This revision is now accepted and ready to land.May 31 2016, 1:45 PM

Do I remember that somebody was working on a more generic expression-like syntax for REQUIRES? If that's been abandoned, then Never Mind.

ddunbar edited edge metadata.May 31 2016, 4:25 PM
ddunbar added a subscriber: ddunbar.

I believe you are thinking of: http://reviews.llvm.org/D18185

EricWF closed this revision.Jun 1 2016, 7:06 PM