This is an archive of the discontinued LLVM Phabricator instance.

[lld][ELF][NFC] Improve AVR tests
ClosedPublic

Authored by benshi001 on Apr 26 2023, 11:11 PM.

Diff Detail

Event Timeline

benshi001 created this revision.Apr 26 2023, 11:11 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 26 2023, 11:11 PM
benshi001 requested review of this revision.Apr 26 2023, 11:11 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 26 2023, 11:11 PM
benshi001 updated this revision to Diff 518410.May 1 2023, 1:36 AM

Other basic-xxx.s tests also run llvm-readobj --file-headers --sections -l --symbols.

MaskRay added inline comments.May 1 2023, 10:03 AM
lld/test/ELF/basic-avr.s
5

Prefer llvm-readelf to llvm-readobj for section and program header dumping.

benshi001 updated this revision to Diff 518614.May 1 2023, 6:25 PM
benshi001 marked an inline comment as done.
MaskRay accepted this revision.May 1 2023, 6:31 PM

Prefer [test] to [NFC] for pure-test patches.

(Personally I don't use the project tag ([lld]). The component tag seems more useful.

This revision is now accepted and ready to land.May 1 2023, 6:31 PM

Prefer [test] to [NFC] for pure-test patches.

(Personally I don't use the project tag ([lld]). The component tag seems more useful.

I will change that when committing. Thanks!

This revision was automatically updated to reflect the committed changes.