You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@teaclave.apache.org by Frank Denis <no...@github.com.INVALID> on 2021/08/23 21:57:57 UTC

[apache/incubator-teaclave] Doc: C code can also be compiled using Zig (#550)

## Description

No need for another full LLVM installation if Zig is already installed.

Fixes # (issue)

## Type of change (select or add applied and delete the others)

- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [X] API change with a documentation update
- [ ] Additional test coverage
- [ ] Code cleanup or just sync with upstream third-party crates

## How has this been tested?

By compiling the example code using `zig` instead of installing `wasi-sdk`.

## Checklist

- [X] Fork the repo and create your branch from `master`.
- [ ] If you&#39;ve added code that should be tested, add tests.
- [ ] If you&#39;ve changed APIs, update the documentation.
- [ ] Ensure the tests pass (see CI results).
- [ ] Make sure your code lints/format.

You can view, comment on, or merge this pull request online at:

  https://github.com/apache/incubator-teaclave/pull/550

-- Commit Summary --

  * Doc: C code can also be compiled using Zig

-- File Changes --

    M docs/executing-wasm.md (7)

-- Patch Links --

https://github.com/apache/incubator-teaclave/pull/550.patch
https://github.com/apache/incubator-teaclave/pull/550.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/apache/incubator-teaclave/pull/550

Re: [apache/incubator-teaclave] Doc: C code can also be compiled using Zig (#550)

Posted by Mingshen Sun <no...@github.com.INVALID>.
> I tried `zig` 0.8 on Ubuntu and it works for `simple_add` example.
> 
> I'm not so familiar with `zig`, @jedisct1 could you please explain the benefits of compiling C to WASM with `zig`? It would be great if you can add one or two sentences in the document!

I believe on benefit he mentioned in the PR is "No need for another full LLVM installation if Zig is already installed."

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/apache/incubator-teaclave/pull/550#issuecomment-904303114

Re: [apache/incubator-teaclave] Doc: C code can also be compiled using Zig (#550)

Posted by Hongbo <no...@github.com.INVALID>.
I tried `zig` 0.8 on Ubuntu and it works for `simple_add` example.

I'm not so familiar with `zig`, @jedisct1 could you please explain the benefits of compiling C to WASM with `zig`? It would be great if you can add one or two sentences in the document!

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/apache/incubator-teaclave/pull/550#issuecomment-904249296

Re: [apache/incubator-teaclave] Doc: C code can also be compiled using Zig (#550)

Posted by Mingshen Sun <no...@github.com.INVALID>.
Merged #550 into master.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/apache/incubator-teaclave/pull/550#event-5202274613

Re: [apache/incubator-teaclave] Doc: C code can also be compiled using Zig (#550)

Posted by Mingshen Sun <no...@github.com.INVALID>.
Let me try if it works in the WAMR executor. Thanks!

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/apache/incubator-teaclave/pull/550#issuecomment-904169498

Re: [apache/incubator-teaclave] Doc: C code can also be compiled using Zig (#550)

Posted by Matt Sicker <no...@github.com.INVALID>.
Zig supports all of C along with its own Zig programming language. It's a drop in compiler replacement with friendly errors.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/apache/incubator-teaclave/pull/550#issuecomment-904698761