1649631 - wasi issues in clang-11 builds

Status: RESOLVED FIXED

(Firefox Build System :: Toolchains, defect, P3)

Description

[(Away)]

I'm attempting to stand up some builds with clang trunk, currently version 11.

In version 11, the clang tree has a breaking change to wasm which requires that we pick up the corresponding change in wasi-libc which is available in wasi-sdk release 10.

However, when I update the rev for the wasi-sysroot task, the builds fail with:

 LucetcError { inner: ErrorMessage { msg: "Unknown module for symbol `wasi_snapshot_preview1::proc_exit`" }

From this point, I'm not super familiar, so this is just a guess, but: I notice that this symbol appeared in a recent change to the bytecodealliance/lucet repo. Does the PLSysSec fork need to pick it up?

Comment 1

[Nathan Froyd [:froydnj]]

(In reply to :dmajor from comment #0)

From this point, I'm not super familiar, so this is just a guess, but: I notice that this symbol appeared in a recent change to the bytecodealliance/lucet repo. Does the PLSysSec fork need to pick it up?

My understanding is that yes, we would need to update to pick this symbol up. Shravan, is that correct?

Comment 2

[(Away)]

Ping. LLVM 11 is now in release testing and I've had to disable wasm sandboxing in those builds. If there are any lurking compiler issues related to the wasi interaction, it would be much better if we could find and report them before the final release.

Comment 3

[Shravan Narayan]

I apologize for the long overdue response. Have effectively been out of commission for a couple of weeks. I will investigate this weekend and will update the fork.

Comment 4

[Shravan Narayan]

Attachment: Bug 1649631 - Fix wasi issues in clang-11 builds r=froydnj,dmajor

Comment 5

[Shravan Narayan]

Just submitted the patch for this.
@dmajor, froydnj: if one of you could review and land this patch, that would be great.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/99b9eff39f1b
Fix wasi issues in clang-11 builds r=froydnj,dmajor

Comment 7

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/99b9eff39f1b