Skip to content
This repository was archived by the owner on Aug 10, 2021. It is now read-only.

Update LLVM to 6.0.1 and WebAssembly target refactoring #1788

Merged
merged 7 commits into from
Aug 14, 2018
Merged

Conversation

sbogolepov
Copy link
Contributor

WebAssembly improvements:

  • Remove binaryen dependency
  • Faster compilation time
  • Simplify morecore function
  • Simplify launcher.js

@@ -54,7 +54,7 @@ osVersionMin.macos_x64 = 10.11
entrySelector.macos_x64 = -alias _Konan_main _main
dependencies.macos_x64 = \
libffi-3.2.1-2-darwin-macos \
clang-llvm-5.0.0-darwin-macos
clang-llvm-6.0.0-darwin-macos
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

6.0.1?

target-sysroot-2-wasm

targetToolchain.mingw_x64-zephyr_stm32f4_disco = gcc-arm-none-eabi-7-2017-q4-major-win32/arm-none-eabi
dependencies.mingw_x64-zephyr_stm32f4_disco = \
msys2-mingw-w64-x86_64-gcc-7.2.0-clang-llvm-5.0.0-windows-x86-64 \
mingw64 \
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not a very good name.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Execution failed for task ':dependencies:gcc-arm-none-eabi-7-2017-q4-major-win32'.
> org.jetbrains.kotlin.konan.util.DependencyDownloader$HTTPResponseException: Server returned HTTP response code: 403 for URL: https://cache-redirector.jetbrains.com/download.jetbrains.com/kotlin/native/mingw64.zip

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@msink, it's not ready for use yet :)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will it fix #1734 ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, it should.

@@ -1,6 +1,10 @@
#include <math.h>
#include <stdint.h>

#ifdef KONAN_WASM
#include "../Common.h"
Copy link
Contributor

@olonho olonho Jul 10, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please don't use includes by relative paths.
Also add RUNTIME_USED in separate ifdef, code is unreadable this way.

@sbogolepov sbogolepov changed the title Update LLVM to 6.0.1 and WebAssembly target refactoring [WIP] Update LLVM to 6.0.1 and WebAssembly target refactoring Jul 11, 2018
@sbogolepov sbogolepov force-pushed the llvm6 branch 2 times, most recently from d30a0b3 to 7ecdc2b Compare July 18, 2018 10:12
@sbogolepov sbogolepov force-pushed the llvm6 branch 2 times, most recently from edbd0bd to 84d1b5a Compare August 13, 2018 13:14
@sbogolepov sbogolepov changed the title [WIP] Update LLVM to 6.0.1 and WebAssembly target refactoring Update LLVM to 6.0.1 and WebAssembly target refactoring Aug 13, 2018
@viteinfinite
Copy link

@olonho Does this allow generating valid bitcode (cf #1564)?
Thank you!

@sbogolepov
Copy link
Contributor Author

@viteinfinite this PR has nothing to do with bitcode embedding.

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants