This is an archive of the discontinued LLVM Phabricator instance.

[DFSan] Fix overuse of REQUIRES: in tests.
AbandonedPublic

Authored by browneee on Dec 28 2022, 1:39 PM.

Details

Reviewers
MaskRay
Summary

Following suggestion on D140690.

Diff Detail

Event Timeline

browneee created this revision.Dec 28 2022, 1:39 PM
Herald added a project: Restricted Project. · View Herald TranscriptDec 28 2022, 1:39 PM
browneee requested review of this revision.Dec 28 2022, 1:39 PM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptDec 28 2022, 1:39 PM
Herald added subscribers: Restricted Project, cfe-commits. · View Herald Transcript
browneee updated this revision to Diff 485547.Dec 28 2022, 2:25 PM

Fix updated line numbers in test.

MaskRay accepted this revision.Dec 28 2022, 2:36 PM

Looks great!

This revision is now accepted and ready to land.Dec 28 2022, 2:36 PM
browneee updated this revision to Diff 485552.Dec 28 2022, 3:50 PM

Fix other line number.

browneee abandoned this revision.Dec 29 2022, 2:44 PM

Abandon due to D140744 being submitted.

MaskRay added inline comments.Dec 29 2022, 3:09 PM
clang/lib/Driver/ToolChains/Linux.cpp
766

This change is still useful and worth submitting :)