This is an archive of the discontinued LLVM Phabricator instance.

[llvm] Inclusive language: replace master with main in file paths in LIT tests
ClosedPublic

Authored by quinnp on Nov 4 2021, 7:26 AM.

Details

Summary

[NFC] As part of using inclusive language within the llvm project, this patch
replaces master with main in file/directory paths in llvm LIT tests.

Diff Detail

Event Timeline

quinnp created this revision.Nov 4 2021, 7:26 AM
quinnp requested review of this revision.Nov 4 2021, 7:26 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 4 2021, 7:26 AM
bjope added a comment.Nov 4 2021, 7:36 AM

Suggesting some edits (there shouldn't be a need to have a DIFile directory path (at least not in some of those test cases). So I think it is better to clean it up a bit more if touching those test cases.

llvm/test/DebugInfo/MIR/X86/live-debug-vars-unused-arg-debugonly.mir
60
llvm/test/DebugInfo/MIR/X86/live-debug-vars-unused-arg.mir
58
llvm/test/DebugInfo/X86/sdag-dangling-dbgvalue.ll
144
dmgreen added a subscriber: dmgreen.Nov 4 2021, 7:40 AM
dmgreen added inline comments.
llvm/test/MC/ARM/inline-asm-srcloc.ll
3–4

I think you may be able to just remove these lines. So long as the test still passes, that would be a good cleanup for the test.

quinnp updated this revision to Diff 384756.Nov 4 2021, 8:01 AM

Adressing review comments.

quinnp marked 4 inline comments as done.Nov 4 2021, 8:05 AM

Thanks @bjope and @dmgreen! I've addressed your comments.

bjope accepted this revision.Nov 8 2021, 5:05 AM

LGTM

This revision is now accepted and ready to land.Nov 8 2021, 5:05 AM