This is an archive of the discontinued LLVM Phabricator instance.

[lld/ELF] Convert undef-spell-corrector.s test to split-file
ClosedPublic

Authored by thakis on Oct 13 2022, 7:56 AM.

Diff Detail

Event Timeline

thakis created this revision.Oct 13 2022, 7:56 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 13 2022, 7:56 AM
thakis requested review of this revision.Oct 13 2022, 7:56 AM

Makes the test a bit longer, but makes it a bit easier to find the asm snippets. What do you think?

thakis updated this revision to Diff 467481.Oct 13 2022, 8:21 AM

fix case of a file

MaskRay accepted this revision.Oct 13 2022, 12:37 PM
MaskRay added inline comments.
lld/test/ELF/undef-spell-corrector.s
3

Optional: if you use && cd %t, then you can omit %t/ below.

This revision is now accepted and ready to land.Oct 13 2022, 12:37 PM

Thanks!

lld/test/ELF/undef-spell-corrector.s
3

I don't love that style, as it makes it harder to copy failing run lines.

(My workflow for failing tests:

  1. Run individual failing test with bin/llvm-lit -vv path/to/test
  2. Copy failing run line to terminal
  3. Go from there

Maybe there's some better approach?)

Herald added a project: Restricted Project. · View Herald TranscriptOct 14 2022, 6:07 AM