This is an archive of the discontinued LLVM Phabricator instance.

[lld][WebAsssembly] Convert more tests from IR to assembly. NFC
ClosedPublic

Authored by sbc100 on Jul 28 2022, 5:41 PM.

Diff Detail

Event Timeline

sbc100 created this revision.Jul 28 2022, 5:41 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 28 2022, 5:41 PM
Herald added subscribers: pmatos, asb, wingo. · View Herald Transcript
sbc100 requested review of this revision.Jul 28 2022, 5:41 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 28 2022, 5:41 PM
dschuff accepted this revision.Jul 29 2022, 11:08 AM

LGTM.
As we move more of ld tests to .s I assume (hope?) we have corresponding tests that cover the IR part of the translation, e.g. testing that extern weak global in IR results in .weak in the assembly...

This revision is now accepted and ready to land.Jul 29 2022, 11:08 AM

LGTM.
As we move more of ld tests to .s I assume (hope?) we have corresponding tests that cover the IR part of the translation, e.g. testing that extern weak global in IR results in .weak in the assembly...

Yes, there should be nothing in the IR that gets tested here under lld. That would be really odd/wrong.

In the case of .weak we have llvm/test/CodeGen/WebAssembly/weak.ll

lld/test/wasm/archive-weak-undefined.s