This is an archive of the discontinued LLVM Phabricator instance.

[lld/tests] Stop setting the "asserts" and "debug" features
ClosedPublic

Authored by thakis on Dec 17 2021, 7:07 AM.

Details

Summary

The last use of REQUIRES: asserts was removed in 2015 in 251b0e268 when the
old COFF linker was removed.

The last use of REQUIRES: debug was removed in 2013 in 72c5d3d7c in favor of
REQUIRES: asserts.

lld's test suite currently has no behavior difference with respect to
assertions or debug builds (and hasn't had it for 6 years). Let's keep it that
way :)

Diff Detail

Event Timeline

thakis requested review of this revision.Dec 17 2021, 7:07 AM
thakis created this revision.
MaskRay accepted this revision.Jan 4 2022, 5:01 PM

Looks great!

This revision is now accepted and ready to land.Jan 4 2022, 5:01 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 5 2022, 10:39 AM