This is an archive of the discontinued LLVM Phabricator instance.

[lld][WebAssembly] Convert more tests to asm format. NFC.
ClosedPublic

Authored by sbc100 on Nov 17 2020, 10:08 PM.

Diff Detail

Event Timeline

sbc100 created this revision.Nov 17 2020, 10:08 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 17 2020, 10:08 PM
sbc100 requested review of this revision.Nov 17 2020, 10:08 PM
sbc100 updated this revision to Diff 306150.Nov 18 2020, 10:26 AM
  • update tests
dschuff accepted this revision.Nov 18 2020, 2:00 PM

This conversion LGTM.
It occurred to me that I don't know if we have tests that actually cover the conversion that we're removing here, namely LLVM IR -> assembly. e.g. after a really quick search I didn't see an ll test testing extern_weak globals in llvm/test/CodeGen/WebAssembly/ or llvm/test/MC/WebAssembly/. Did I just miss it?

This revision is now accepted and ready to land.Nov 18 2020, 2:00 PM

This conversion LGTM.
It occurred to me that I don't know if we have tests that actually cover the conversion that we're removing here, namely LLVM IR -> assembly. e.g. after a really quick search I didn't see an ll test testing extern_weak globals in llvm/test/CodeGen/WebAssembly/ or llvm/test/MC/WebAssembly/. Did I just miss it?

Good question. I do normally check when I do these conversions but maybe I forgot this time. I will double check before I land.

sbc100 updated this revision to Diff 306553.Nov 19 2020, 4:35 PM

A codegen test

dschuff accepted this revision.Nov 19 2020, 4:37 PM
dschuff added inline comments.
llvm/test/CodeGen/WebAssembly/weak.ll
19

nit: extra space before CHECK

Added a new CodeGen test to covert extern_weak

sbc100 added inline comments.Nov 19 2020, 4:54 PM
llvm/test/CodeGen/WebAssembly/weak.ll
19

I've been using this technique in the lld tests to that the : chars line up and allow expectations to be more easily read IMHO. But this is a different place so I will change.

sbc100 updated this revision to Diff 306557.Nov 19 2020, 4:55 PM
  • feedback
This revision was landed with ongoing or failed builds.Nov 19 2020, 4:57 PM
This revision was automatically updated to reflect the committed changes.
lld/test/wasm/weak-symbols.s