This is an archive of the discontinued LLVM Phabricator instance.

[DebugInfo][test][NFC] Remove case numbers
ClosedPublic

Authored by jhenderson on Feb 11 2020, 6:17 AM.

Details

Summary

This reduces the noise caused by adding cases earlier in the sequence.

Diff Detail

Event Timeline

jhenderson created this revision.Feb 11 2020, 6:17 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 11 2020, 6:17 AM
dblaikie accepted this revision.Feb 11 2020, 9:52 AM

Looks good (should these tests be testing the diagnostic text of the recoverable errors, in addition to testing the fact that they are recovered from/dumped?)

This revision is now accepted and ready to land.Feb 11 2020, 9:52 AM

Looks good (should these tests be testing the diagnostic text of the recoverable errors, in addition to testing the fact that they are recovered from/dumped?)

They do - see the block at the bottom of the test with the ALL/OTHER prefixes. They aren't inline with the body, as I don't think the exact interleaving of the output is necessarily that important.

This revision was automatically updated to reflect the committed changes.