This is an archive of the discontinued LLVM Phabricator instance.

[demangler][NFC] Reformat test part 1
AbandonedPublic

Authored by urnathan on Feb 18 2022, 5:27 AM.

Details

Summary

The CI linter complains about the formatting in later changes. As the diff is too large for phabricator, this is part 1

Diff Detail

Event Timeline

urnathan created this revision.Feb 18 2022, 5:27 AM
urnathan requested review of this revision.Feb 18 2022, 5:27 AM

Could the test be tagged with keywords that would prevent the linting to kick in? The amount of changes for this and following patches seems too big to be worth the reformatting.

Could the test be tagged with keywords that would prevent the linting to kick in? The amount of changes for this and following patches seems too big to be worth the reformatting.

I don't know. but I've done the reformatting now. and threaded it through phabricator's limits.

ChuanqiXu added a comment.EditedFeb 20 2022, 6:15 PM

According to my understanding to LLVM policy, we could land such patches (NFC and test only) directly without review.

urnathan abandoned this revision.Mar 3 2022, 6:43 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 3 2022, 6:43 AM