This is an archive of the discontinued LLVM Phabricator instance.

[WebAssembly] Implement __builtin_wasm_tls_base intrinsic
ClosedPublic

Authored by quantum on Jul 17 2019, 5:38 PM.

Details

Event Timeline

quantum created this revision.Jul 17 2019, 5:38 PM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptJul 17 2019, 5:38 PM
aheejin accepted this revision.Jul 17 2019, 6:18 PM

What does this return when __wasm_init_tls has not been called?

This revision is now accepted and ready to land.Jul 17 2019, 6:18 PM

What does this return when __wasm_init_tls has not been called?

It returns 0, which I think is fine.

This revision was automatically updated to reflect the committed changes.
sunfish added inline comments.Jul 18 2019, 12:19 PM
cfe/trunk/include/clang/Basic/BuiltinsWebAssembly.def
34 ↗(On Diff #210632)

This can also be "p" for "pure", which despite the name, just means there are no mutating side effects.