This is an archive of the discontinued LLVM Phabricator instance.

[NFC] Disallow unused prefixes in clang/test/Analysis
ClosedPublic

Authored by mtrofin on Jan 22 2021, 10:32 AM.

Diff Detail

Event Timeline

mtrofin requested review of this revision.Jan 22 2021, 10:32 AM
mtrofin created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptJan 22 2021, 10:32 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
Charusso edited reviewers, added: NoQ; removed: dergachev.a.Jan 22 2021, 10:35 AM
NoQ added a comment.Jan 22 2021, 7:22 PM

I added those for consistency but i think your effort is worth it so let's ditch them.

clang/test/Analysis/exploded-graph-rewriter/trimmers.dot
4

Uh-oh, these look like typos as there are lines for NOTTHREE and NOTFOUR. Looks like your effort has paid off and found a real bug in tests! The tests seem to be passing just fine after i change to NOTTHREE and NOTFOUR respectively.

mtrofin updated this revision to Diff 319024.Jan 25 2021, 8:31 AM

fixed trimmers.dot

NoQ accepted this revision.Jan 25 2021, 12:08 PM

Thanks!!

This revision is now accepted and ready to land.Jan 25 2021, 12:08 PM
This revision was automatically updated to reflect the committed changes.