This is an archive of the discontinued LLVM Phabricator instance.

[WebAssembly] Add TLS FileCheck lines to Wasm SjLj test
ClosedPublic

Authored by aheejin on Feb 14 2022, 6:08 PM.

Diff Detail

Event Timeline

aheejin created this revision.Feb 14 2022, 6:08 PM
aheejin requested review of this revision.Feb 14 2022, 6:08 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 14 2022, 6:08 PM
aheejin retitled this revision from [WebAssembly] Add TLS FileCheck lines to Wasm SjLj to [WebAssembly] Add TLS FileCheck lines to Wasm SjLj test.Feb 14 2022, 6:08 PM
aheejin added inline comments.
llvm/test/CodeGen/WebAssembly/lower-wasm-sjlj.ll
20

In TLS mode, this line contains #0, which makes coming up with a CHECK line that works for both TLS and non-TLS annoying, and I don't think this line worth making two separate TLS and NO-TLS check lines, so I'm cutting this off.

sbc100 accepted this revision.Feb 15 2022, 8:32 AM
This revision is now accepted and ready to land.Feb 15 2022, 8:32 AM
This revision was landed with ongoing or failed builds.Feb 15 2022, 3:52 PM
This revision was automatically updated to reflect the committed changes.