[libcxxabi] [libunwind] [WebAssembly] __USING_WASM_EXCEPTIONS__ -> __WASM_EXCEPTIONS__ (PR #92840)

2024-05-21 Thread Sam Clegg via cfe-commits
sbc100 wrote: > It is hard to fix the clang-format errors because the modified files > generally don't conform to the clang-format style anyway, so clang-format my > change currently affects hundreds of unrelated lines. > > ``` > aheejin@aheejin:~/llvm-project/libunwind$ git clang-format main

[libcxxabi] [libunwind] [WebAssembly] __USING_WASM_EXCEPTIONS__ -> __WASM_EXCEPTIONS__ (PR #92840)

2024-05-21 Thread Sam Clegg via cfe-commits
https://github.com/sbc100 approved this pull request. https://github.com/llvm/llvm-project/pull/92840 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [WebAssembly] Define __WASM_EXCEPTIONS__ for -fwasm-exceptions (PR #92604)

2024-05-17 Thread Sam Clegg via cfe-commits
https://github.com/sbc100 approved this pull request. lgtm. @dschuff WDYT? https://github.com/llvm/llvm-project/pull/92604 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [WebAssembly] Define __USING_WASM_EXCEPTIONS__ for -fwasm-exceptions (PR #92604)

2024-05-17 Thread Sam Clegg via cfe-commits
@@ -1006,6 +1006,8 @@ static void InitializePredefinedMacros(const TargetInfo , else if (LangOpts.hasDWARFExceptions() && (TI.getTriple().isThumb() || TI.getTriple().isARM())) Builder.defineMacro("__ARM_DWARF_EH__"); + else if (LangOpts.hasWasmExceptions()

[clang] [WebAssembly] Define __USING_WASM_EXCEPTIONS__ for -fwasm-exceptions (PR #92604)

2024-05-17 Thread Sam Clegg via cfe-commits
https://github.com/sbc100 edited https://github.com/llvm/llvm-project/pull/92604 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [WebAssembly] Define __USING_WASM_EXCEPTIONS__ for -fwasm-exceptions (PR #92604)

2024-05-17 Thread Sam Clegg via cfe-commits
@@ -1006,6 +1006,8 @@ static void InitializePredefinedMacros(const TargetInfo , else if (LangOpts.hasDWARFExceptions() && (TI.getTriple().isThumb() || TI.getTriple().isARM())) Builder.defineMacro("__ARM_DWARF_EH__"); + else if (LangOpts.hasWasmExceptions()

[clang] [WebAssembly] Define __USING_WASM_EXCEPTIONS__ for -fwasm-exceptions (PR #92604)

2024-05-17 Thread Sam Clegg via cfe-commits
@@ -1006,6 +1006,8 @@ static void InitializePredefinedMacros(const TargetInfo , else if (LangOpts.hasDWARFExceptions() && (TI.getTriple().isThumb() || TI.getTriple().isARM())) Builder.defineMacro("__ARM_DWARF_EH__"); + else if (LangOpts.hasWasmExceptions()

[clang] [clang-tools-extra] Reapply "[Clang][Sema] Diagnose class member access expressions naming non-existent members of the current instantiation prior to instantiation in the absence of dependent

2024-05-02 Thread Sam Clegg via cfe-commits
sbc100 wrote: This change recently rolled into the emscripten SDK and seems to be breaking the build of regal. I've not invistigated this yet, and its not our code: ``` em++ -c /usr/local/google/home/sbc/dev/wasm/emscripten/cache/ports/regal/regal-version_7/src/regal/RegalIff.cpp -o

[clang] [llvm] [Pipelines][Coroutines] Tune coroutine passes only for ThinLTO pre-link pipeline (PR #90690)

2024-04-30 Thread Sam Clegg via cfe-commits
https://github.com/sbc100 approved this pull request. I'm not very familiar with this code, but it looks reasonable on the surface. https://github.com/llvm/llvm-project/pull/90690 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[clang] [llvm] [Pipelines] Do not run CoroSplit and CoroCleanup in LTO pre-link pipeline (PR #90310)

2024-04-30 Thread Sam Clegg via cfe-commits
sbc100 wrote: The function being compiled in the crash looks like this one: https://github.com/emscripten-core/emscripten/blob/56547157a37e25fb31f9193461a2c88bdaa44385/system/lib/embind/bind.cpp#L66-L68 https://github.com/llvm/llvm-project/pull/90310

[clang] [llvm] [Pipelines] Do not run CoroSplit and CoroCleanup in LTO pre-link pipeline (PR #90310)

2024-04-30 Thread Sam Clegg via cfe-commits
sbc100 wrote: This change seems to causing the auto-roller for emscripten to fail due to a crash in LTO. See https://ci.chromium.org/ui/p/emscripten-releases/builders/luci.emscripten-releases.ci/linux-test-suites ``` PromoteIntegerOperand Op #3: t22: i32,ch = llvm.coro.subfn.addr t26,

[clang] [Driver] Don't default to -mrelax-all for non-RISCV -O0 (PR #90013)

2024-04-26 Thread Sam Clegg via cfe-commits
sbc100 wrote: This change seems to have caused a test in emscripten to starting failing with `argument unused during compilation: '-O2'`: https://github.com/emscripten-core/emscripten/pull/21841. Is that an intended side effect of this change? The fix on our end seems reasonable but posting

[clang] [WebAssembly] Enable multivalue and reference-types in generic CPU config (PR #80923)

2024-04-23 Thread Sam Clegg via cfe-commits
@@ -147,19 +147,25 @@ void WebAssemblyTargetInfo::setFeatureEnabled(llvm::StringMap , bool WebAssemblyTargetInfo::initFeatureMap( llvm::StringMap , DiagnosticsEngine , StringRef CPU, const std::vector ) const { - if (CPU == "bleeding-edge") { -

[clang] [WebAssembly] Enable multivalue and reference-types in generic CPU config (PR #80923)

2024-04-23 Thread Sam Clegg via cfe-commits
sbc100 wrote: After this lands does that mean that all object files will have `multi-value` in the features section? Or only object files that actually use `mutli-value`? https://github.com/llvm/llvm-project/pull/80923 ___ cfe-commits mailing

[clang] [llvm] [WebAssembly] Implement an alternative translation for -wasm-enable-sjlj (PR #84137)

2024-03-25 Thread Sam Clegg via cfe-commits
sbc100 wrote: Is this good to land now? @yamt do you have commit access or should @aheejin or myself land this? https://github.com/llvm/llvm-project/pull/84137 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[clang] [WebAssembly] Change the default linker for `wasm32-wasip2` (PR #84569)

2024-03-18 Thread Sam Clegg via cfe-commits
https://github.com/sbc100 approved this pull request. https://github.com/llvm/llvm-project/pull/84569 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [WebAssembly] Implement an alternative translation for -wasm-enable-sjlj (PR #84137)

2024-03-13 Thread Sam Clegg via cfe-commits
@@ -198,9 +198,18 @@ /// /// If there are calls to setjmp() /// -/// 2) and 3): The same as 2) and 3) in Emscripten SjLj. -/// (setjmpTable/setjmpTableSize initialization + setjmp callsite -/// transformation) +/// 2) In the function entry that calls setjmp, initialize +///

[clang] [llvm] [WebAssembly] Implement an alternative translation for -wasm-enable-sjlj (PR #84137)

2024-03-13 Thread Sam Clegg via cfe-commits
https://github.com/sbc100 commented: Let me know if you want some help testing alsongside the emscripten side. Alternatively myself or @aheejin could perform the emscripten tests. https://github.com/llvm/llvm-project/pull/84137 ___ cfe-commits

[clang] [llvm] [WebAssembly] Implement an alternative translation for -wasm-enable-sjlj (PR #84137)

2024-03-13 Thread Sam Clegg via cfe-commits
https://github.com/sbc100 edited https://github.com/llvm/llvm-project/pull/84137 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [WebAssembly] Implement an alternative translation for -wasm-enable-sjlj (PR #84137)

2024-03-13 Thread Sam Clegg via cfe-commits
@@ -198,9 +198,18 @@ /// /// If there are calls to setjmp() /// -/// 2) and 3): The same as 2) and 3) in Emscripten SjLj. -/// (setjmpTable/setjmpTableSize initialization + setjmp callsite -/// transformation) +/// 2) In the function entry that calls setjmp, initialize +///

[clang] [llvm] [WebAssembly] Implement an alternative translation for -wasm-enable-sjlj (PR #84137)

2024-03-11 Thread Sam Clegg via cfe-commits
sbc100 wrote: > @aheejin @sbc100 let me confirm the plan on this PR. i can remove the option > `-mllvm -experimental-wasm-enable-alt-sjlj` by making it unconditionally > true, and update tests, right? That is my understanding yes. https://github.com/llvm/llvm-project/pull/84137

[clang] [llvm] [WebAssembly] Implement an alternative translation for -wasm-enable-sjlj (PR #84137)

2024-03-09 Thread Sam Clegg via cfe-commits
sbc100 wrote: > > In terms of getting this landed and tested, I wonder which path we should > > take: > > > > 1. Land this now, without tests, then update emscripten then come back and > > flip the default, at which point the existing tests will get updated. > > 2. Duplicate/update the the

[clang] [WebAssembly] Change the default linker for `wasm32-wasip2` (PR #84569)

2024-03-08 Thread Sam Clegg via cfe-commits
sbc100 wrote: > Currently it accepts no extra inputs, but in the future I'd expect that it'll > grow an option or two of its own I was more asking about whether the file types is accepts are anything more than object files and libraries. i.e. can you pass other core modules and have

[clang] [WebAssembly] Change the default linker for `wasm32-wasip2` (PR #84569)

2024-03-08 Thread Sam Clegg via cfe-commits
sbc100 wrote: I see, so `wasm-ld` builds "core module + metadata" and then `wasm-component-ld` takes that metadata and uses it to wrap the core module into a component (with exactly that one core module inside of it). So the core module + metadata is kind of isomorphic with that

[clang] [WebAssembly] Change the default linker for `wasm32-wasip2` (PR #84569)

2024-03-08 Thread Sam Clegg via cfe-commits
sbc100 wrote: Regarding WebAssembly/wasi-sdk and WebAssembly/wasi-libc, is there any reason why simple programs wouldn't be core modules? Won't most C/C++ programs still be build-able as just core modules? https://github.com/llvm/llvm-project/pull/84569

[clang] [WebAssembly] Change the default linker for `wasm32-wasip2` (PR #84569)

2024-03-08 Thread Sam Clegg via cfe-commits
https://github.com/sbc100 commented: Does wasm-component-ld accept any other input types other than the ones that wasm-ld accepts? Does wasm-component-ld call wasm-ld internally? Do we expect clang users to be building compound components using a single clang command? i.e. will they be

[clang] [llvm] [WebAssembly] Implement an alternative translation for -wasm-enable-sjlj (PR #84137)

2024-03-08 Thread Sam Clegg via cfe-commits
https://github.com/sbc100 commented: In terms of getting this landed and tested, I wonder which path we should take: 1. Land this now, without tests, then update emscripten then come back and flip the default, at which point the existing tests will get updated. 2. Duplicate/update the the

[clang] [llvm] [WebAssembly] Implement an alternative translation for -wasm-enable-sjlj (PR #84137)

2024-03-07 Thread Sam Clegg via cfe-commits
@@ -999,25 +1002,42 @@ bool WebAssemblyLowerEmscriptenEHSjLj::runOnModule(Module ) { // Register __wasm_longjmp function, which calls __builtin_wasm_longjmp. FunctionType *FTy = FunctionType::get( IRB.getVoidTy(), {Int8PtrTy, IRB.getInt32Ty()}, false); -

[clang] [llvm] [WebAssembly] Implement an alternative translation for -wasm-enable-sjlj (PR #84137)

2024-03-07 Thread Sam Clegg via cfe-commits
https://github.com/sbc100 edited https://github.com/llvm/llvm-project/pull/84137 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [WebAssembly] Implement an alternative translation for -wasm-enable-sjlj (PR #84137)

2024-03-07 Thread Sam Clegg via cfe-commits
https://github.com/sbc100 commented: Thanks for working on this! I'm excited to see this land and become less emscripten-specific. I'm hoping that once we get the emscripten side landed we can remove the new option very soon too. https://github.com/llvm/llvm-project/pull/84137

[clang] [WebAssembly] Add more features to generic CPU config (PR #80923)

2024-02-06 Thread Sam Clegg via cfe-commits
sbc100 wrote: > Tried running Emscripten tests, and some of them fail with reference-types or > bulk-memory. I haven't investigated the causes yet, but it could be better to > start with the other two (nontrapping-fptoint and multivalue). No need to block these on emscripten failures.. I'm

[clang] [WebAssembly] Add more features to generic CPU config (PR #80923)

2024-02-06 Thread Sam Clegg via cfe-commits
sbc100 wrote: @kripken https://github.com/llvm/llvm-project/pull/80923 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[llvm] [clang-tools-extra] [clang] [lld] [Object][Wasm] Allow parsing of GC types in type and table sections (PR #79235)

2024-01-24 Thread Sam Clegg via cfe-commits
https://github.com/sbc100 approved this pull request. https://github.com/llvm/llvm-project/pull/79235 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[lld] [clang] [llvm] [clang-tools-extra] [Object][Wasm] Allow parsing of GC types in type and table sections (PR #79235)

2024-01-24 Thread Sam Clegg via cfe-commits
@@ -259,10 +259,13 @@ class InputFunction : public InputChunk { file->codeSection->Content.slice(inputSectionOffset, function->Size); debugName = function->DebugName; comdat = function->Comdat; +assert(s.Kind != WasmSignature::Placeholder);

[lld] [clang] [llvm] Embed the command line arguments during LTO (PR #79390)

2024-01-24 Thread Sam Clegg via cfe-commits
@@ -40,6 +41,7 @@ using namespace llvm; namespace lld::wasm { static std::unique_ptr createLTO() { lto::Config c; + c.EmbedCmdArgs = commonContext().cmdArgs; sbc100 wrote: I think it might be the other way around. It looks like only COFF currently

[llvm] [lld] [clang] Embed the command line arguments during LTO (PR #79390)

2024-01-24 Thread Sam Clegg via cfe-commits
@@ -40,6 +41,7 @@ using namespace llvm; namespace lld::wasm { static std::unique_ptr createLTO() { lto::Config c; + c.EmbedCmdArgs = commonContext().cmdArgs; sbc100 wrote: (I'm not sure why `context()` needs to take a template param, and it looks like its

[llvm] [lld] [clang] Embed the command line arguments during LTO (PR #79390)

2024-01-24 Thread Sam Clegg via cfe-commits
@@ -40,6 +41,7 @@ using namespace llvm; namespace lld::wasm { static std::unique_ptr createLTO() { lto::Config c; + c.EmbedCmdArgs = commonContext().cmdArgs; sbc100 wrote: How about adding a helper like the existing ones `CommonLinkerContext.h`: ```

[lld] [clang] [llvm] Embed the command line arguments during LTO (PR #79390)

2024-01-24 Thread Sam Clegg via cfe-commits
@@ -40,6 +41,7 @@ using namespace llvm; namespace lld::wasm { static std::unique_ptr createLTO() { lto::Config c; + c.EmbedCmdArgs = commonContext().cmdArgs; sbc100 wrote: It looks like `commonContext()` is not actually used elsewhere in lld, and this

[clang] [llvm] [clang-tools-extra] [lld] [Object][Wasm] Allow parsing of GC types in type and table sections (PR #79235)

2024-01-24 Thread Sam Clegg via cfe-commits
@@ -259,10 +259,13 @@ class InputFunction : public InputChunk { file->codeSection->Content.slice(inputSectionOffset, function->Size); debugName = function->DebugName; comdat = function->Comdat; +assert(s.Kind != WasmSignature::Placeholder);

[clang] 89d5635 - [lld][WebAssembly] Add --keep-section flag

2023-11-02 Thread Sam Clegg via cfe-commits
Author: Sam Clegg Date: 2023-11-02T14:23:45-07:00 New Revision: 89d5635f0a834e53ac5446c6d6ea38d9166b2a55 URL: https://github.com/llvm/llvm-project/commit/89d5635f0a834e53ac5446c6d6ea38d9166b2a55 DIFF: https://github.com/llvm/llvm-project/commit/89d5635f0a834e53ac5446c6d6ea38d9166b2a55.diff

[clang-tools-extra] Ensure NoTrapAfterNoreturn is false for the wasm backend (PR #65876)

2023-09-27 Thread Sam Clegg via cfe-commits
sbc100 wrote: > But `--no-trap-after-noreturn` didn't exist before, so there was no way to > specify that from the command line. You _created_ it, originally in this PR, > and then the split-off PR in #67051. If this is a bugfix, it sounds like you > are fixing a bug of your own making. I

[clang] Ensure NoTrapAfterNoreturn is false for the wasm backend (PR #65876)

2023-09-27 Thread Sam Clegg via cfe-commits
sbc100 wrote: > But `--no-trap-after-noreturn` didn't exist before, so there was no way to > specify that from the command line. You _created_ it, originally in this PR, > and then the split-off PR in #67051. If this is a bugfix, it sounds like you > are fixing a bug of your own making. I

[clang] [WebAssembly] Update generic CPU configuration to include nontrapping-fptoint (PR #67459)

2023-09-26 Thread Sam Clegg via cfe-commits
sbc100 wrote: Oh I see I think the issue was Safari: https://webassembly.org/roadmap/. Looks like this requires Safari 15 :( https://github.com/llvm/llvm-project/pull/67459 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[clang] [WebAssembly] Update generic CPU configuration to include nontrapping-fptoint (PR #67459)

2023-09-26 Thread Sam Clegg via cfe-commits
https://github.com/sbc100 resolved https://github.com/llvm/llvm-project/pull/67459 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [WebAssembly] Update generic CPU configuration to include nontrapping-fptoint (PR #67459)

2023-09-26 Thread Sam Clegg via cfe-commits
https://github.com/sbc100 updated https://github.com/llvm/llvm-project/pull/67459 >From 64d9d8eb1da9f275cde47bd92c39e600e5a2f9b3 Mon Sep 17 00:00:00 2001 From: Sam Clegg Date: Tue, 26 Sep 2023 09:55:59 -0700 Subject: [PATCH] [WebAssembly] Update generic CPU configuration to include

[clang] [WebAssembly] Update generic CPU configuration to include nontrapping-fptoint (PR #67459)

2023-09-26 Thread Sam Clegg via cfe-commits
sbc100 wrote: @kripken @dschuff @sunfishcode, can you remember why we didn't include this is part of https://reviews.llvm.org/D125728 and https://reviews.llvm.org/D125729? Has the landscape changed enough since then that we can/should land this now?

[clang] [WebAssembly] Update generic CPU configuration to include nontrapping-fptoint (PR #67459)

2023-09-26 Thread Sam Clegg via cfe-commits
https://github.com/sbc100 created https://github.com/llvm/llvm-project/pull/67459 Enable nontrapping-fptoint in -mcpu=generic. This makes this feature enabled by default. This feature is part of [finished proposals], and all major wasm engines support it. [finished proposals]:

[clang] 989ce06 - [clang][WebAssembly] Link crt1 even in case of -shared

2023-08-14 Thread Sam Clegg via cfe-commits
Author: YAMAMOTO Takashi Date: 2023-08-14T09:36:44-07:00 New Revision: 989ce069a4638fd561bebc95f478ca9e45154fec URL: https://github.com/llvm/llvm-project/commit/989ce069a4638fd561bebc95f478ca9e45154fec DIFF:

[clang] 55e199a - [clang][WebAssembly] Support wasm32-wasi shared libraries

2023-06-26 Thread Sam Clegg via cfe-commits
Author: Joel Dice Date: 2023-06-26T10:31:40-07:00 New Revision: 55e199a2c9f4b218499733d60129deffa0a025fe URL: https://github.com/llvm/llvm-project/commit/55e199a2c9f4b218499733d60129deffa0a025fe DIFF: https://github.com/llvm/llvm-project/commit/55e199a2c9f4b218499733d60129deffa0a025fe.diff

[clang] 849df8f - [clang][WebAssembly] Pass `-Wa,--no-type-check` through to the MC layer

2022-09-01 Thread Sam Clegg via cfe-commits
Author: Sam Clegg Date: 2022-09-01T02:56:58-07:00 New Revision: 849df8f6f0a6ded83e36aa52b6daa6b950289804 URL: https://github.com/llvm/llvm-project/commit/849df8f6f0a6ded83e36aa52b6daa6b950289804 DIFF: https://github.com/llvm/llvm-project/commit/849df8f6f0a6ded83e36aa52b6daa6b950289804.diff

[clang] 78747bd - [clang][WebAssembly] Loosen restriction on `main` symbol mangling

2022-06-15 Thread Sam Clegg via cfe-commits
Author: Sam Clegg Date: 2022-06-15T13:56:05-07:00 New Revision: 78747bd39af807eb6b56709b9713c303704c47ad URL: https://github.com/llvm/llvm-project/commit/78747bd39af807eb6b56709b9713c303704c47ad DIFF: https://github.com/llvm/llvm-project/commit/78747bd39af807eb6b56709b9713c303704c47ad.diff

[clang] 47039a1 - [WebAssembly] Remove restriction on main name mangling

2022-06-06 Thread Sam Clegg via cfe-commits
Author: Sam Clegg Date: 2022-06-06T14:04:27-07:00 New Revision: 47039a1a4b299c01dfa0ec531fbc0d24718bb8eb URL: https://github.com/llvm/llvm-project/commit/47039a1a4b299c01dfa0ec531fbc0d24718bb8eb DIFF: https://github.com/llvm/llvm-project/commit/47039a1a4b299c01dfa0ec531fbc0d24718bb8eb.diff

[clang] c832edf - [WebAssembly] Add new target feature in support of 'extended-const' proposal

2022-03-07 Thread Sam Clegg via cfe-commits
Author: Sam Clegg Date: 2022-03-07T16:45:42-08:00 New Revision: c832edfd3fbccfac6d5212d6495f0dde851dbdde URL: https://github.com/llvm/llvm-project/commit/c832edfd3fbccfac6d5212d6495f0dde851dbdde DIFF: https://github.com/llvm/llvm-project/commit/c832edfd3fbccfac6d5212d6495f0dde851dbdde.diff

[clang] ecbcefd - [clang][WebAssemmbly] Call TargetInfo::adjust in derived method.

2022-02-10 Thread Sam Clegg via cfe-commits
Author: Sam Clegg Date: 2022-02-10T16:04:06-08:00 New Revision: ecbcefd693dd62f9242618b3cf838565e829cc7f URL: https://github.com/llvm/llvm-project/commit/ecbcefd693dd62f9242618b3cf838565e829cc7f DIFF: https://github.com/llvm/llvm-project/commit/ecbcefd693dd62f9242618b3cf838565e829cc7f.diff

[clang] 45ad346 - [clang][WebAssembly] Imply -fno-threadsafe-static when threading is disabled

2022-01-31 Thread Sam Clegg via cfe-commits
Author: Sam Clegg Date: 2022-01-31T14:26:09-08:00 New Revision: 45ad3467b708618841869cb529a814001d2295f9 URL: https://github.com/llvm/llvm-project/commit/45ad3467b708618841869cb529a814001d2295f9 DIFF: https://github.com/llvm/llvm-project/commit/45ad3467b708618841869cb529a814001d2295f9.diff

[clang] 1813fde - [WebAssembly] Emit clangast in custom section aligned by 4 bytes

2021-10-19 Thread Sam Clegg via cfe-commits
Author: Yuta Saito Date: 2021-10-19T15:50:08-07:00 New Revision: 1813fde9cc0b56cee42d9b82e6f22fa00a59cdf9 URL: https://github.com/llvm/llvm-project/commit/1813fde9cc0b56cee42d9b82e6f22fa00a59cdf9 DIFF: https://github.com/llvm/llvm-project/commit/1813fde9cc0b56cee42d9b82e6f22fa00a59cdf9.diff

[clang] c05d30e - [clang][Emscripten] Define __unix family of macros

2021-08-25 Thread Sam Clegg via cfe-commits
Author: Sam Clegg Date: 2021-08-25T19:24:47-04:00 New Revision: c05d30e444a1208a8a872002d1146b5cf55e370a URL: https://github.com/llvm/llvm-project/commit/c05d30e444a1208a8a872002d1146b5cf55e370a DIFF: https://github.com/llvm/llvm-project/commit/c05d30e444a1208a8a872002d1146b5cf55e370a.diff

[clang] d1a96e9 - [clang][emscripten] Reduce alignof long double from 16 to 8 bytes

2021-07-02 Thread Sam Clegg via cfe-commits
Author: Sam Clegg Date: 2021-07-02T11:05:40-07:00 New Revision: d1a96e906cc03a95cfd41a1f22bdda92651250c7 URL: https://github.com/llvm/llvm-project/commit/d1a96e906cc03a95cfd41a1f22bdda92651250c7 DIFF: https://github.com/llvm/llvm-project/commit/d1a96e906cc03a95cfd41a1f22bdda92651250c7.diff

[clang] 38a2858 - [clang][emscripten] Add builtin define for __EMSCRIPTEN_PTHREADS__

2021-02-05 Thread Sam Clegg via cfe-commits
Author: Sam Clegg Date: 2021-02-05T13:53:05-08:00 New Revision: 38a285885dd5f3634d7824da2a6aefe5e39782a7 URL: https://github.com/llvm/llvm-project/commit/38a285885dd5f3634d7824da2a6aefe5e39782a7 DIFF: https://github.com/llvm/llvm-project/commit/38a285885dd5f3634d7824da2a6aefe5e39782a7.diff

[clang] 04febd3 - [lld][WebAssembly] Error on import/export of mutable global without `mutable-globals` feature

2020-09-12 Thread Sam Clegg via cfe-commits
Author: Sam Clegg Date: 2020-09-12T14:28:14-07:00 New Revision: 04febd30a8dab3ff4b6e6032f1a1a9f4725f8267 URL: https://github.com/llvm/llvm-project/commit/04febd30a8dab3ff4b6e6032f1a1a9f4725f8267 DIFF: https://github.com/llvm/llvm-project/commit/04febd30a8dab3ff4b6e6032f1a1a9f4725f8267.diff

[clang] 5804a8b - [WebAssebmly] Fully disable 'protected' visibility

2020-06-23 Thread Sam Clegg via cfe-commits
Author: Sam Clegg Date: 2020-06-23T17:50:05-07:00 New Revision: 5804a8b1228ba890d48f4085a3a192ef83c73e00 URL: https://github.com/llvm/llvm-project/commit/5804a8b1228ba890d48f4085a3a192ef83c73e00 DIFF: https://github.com/llvm/llvm-project/commit/5804a8b1228ba890d48f4085a3a192ef83c73e00.diff

[clang] 0731372 - [clang] Allow -DDEFAULT_SYSROOT to be a relative path

2020-03-26 Thread Sam Clegg via cfe-commits
Author: Sam Clegg Date: 2020-03-26T13:48:57-07:00 New Revision: 0731372ee25c8db93e0d976db4be4ffb7c7329c5 URL: https://github.com/llvm/llvm-project/commit/0731372ee25c8db93e0d976db4be4ffb7c7329c5 DIFF: https://github.com/llvm/llvm-project/commit/0731372ee25c8db93e0d976db4be4ffb7c7329c5.diff

[clang] 0a1e349 - [WebAssembly] Setting export_name implies llvm.used

2019-12-16 Thread Sam Clegg via cfe-commits
Author: Sam Clegg Date: 2019-12-16T14:48:38-08:00 New Revision: 0a1e349a7933f7880971533175e11b4bfd22bd53 URL: https://github.com/llvm/llvm-project/commit/0a1e349a7933f7880971533175e11b4bfd22bd53 DIFF: https://github.com/llvm/llvm-project/commit/0a1e349a7933f7880971533175e11b4bfd22bd53.diff

[clang] 881d877 - [WebAssembly] Add new `export_name` clang attribute for controlling wasm export names

2019-12-11 Thread Sam Clegg via cfe-commits
Author: Sam Clegg Date: 2019-12-11T11:54:57-08:00 New Revision: 881d877846e2904c731d616731969421ce8cc825 URL: https://github.com/llvm/llvm-project/commit/881d877846e2904c731d616731969421ce8cc825 DIFF: https://github.com/llvm/llvm-project/commit/881d877846e2904c731d616731969421ce8cc825.diff

r366518 - Fix formatting of inline argument comments. NFC.

2019-07-18 Thread Sam Clegg via cfe-commits
Author: sbc Date: Thu Jul 18 17:30:23 2019 New Revision: 366518 URL: http://llvm.org/viewvc/llvm-project?rev=366518=rev Log: Fix formatting of inline argument comments. NFC. Also, remove the final arg from ItaniumCXXABI in the PNaCl case since its not needed. Differential Revision:

r363759 - git-clang-format: Remove trailing whitespace in docstring. NFC.

2019-06-18 Thread Sam Clegg via cfe-commits
Author: sbc Date: Tue Jun 18 18:52:41 2019 New Revision: 363759 URL: http://llvm.org/viewvc/llvm-project?rev=363759=rev Log: git-clang-format: Remove trailing whitespace in docstring. NFC. Differential Revision: https://reviews.llvm.org/D62915 Modified:

r363241 - [WebAssembly] Modernize include path handling

2019-06-13 Thread Sam Clegg via cfe-commits
Author: sbc Date: Thu Jun 13 02:42:43 2019 New Revision: 363241 URL: http://llvm.org/viewvc/llvm-project?rev=363241=rev Log: [WebAssembly] Modernize include path handling Move include path construction from InitHeaderSearch::AddDefaultIncludePaths in the Driver which appears to be the more

r363238 - [clang-scan-deps] Fix -DBUILD_SHARED_LIBS=ON build

2019-06-13 Thread Sam Clegg via cfe-commits
Author: sbc Date: Thu Jun 13 01:58:46 2019 New Revision: 363238 URL: http://llvm.org/viewvc/llvm-project?rev=363238=rev Log: [clang-scan-deps] Fix -DBUILD_SHARED_LIBS=ON build The -DBUILD_SHARED_LIBS=ON build was broken in rL363204 Differential Revision: https://reviews.llvm.org/D63245

r362959 - [WebAssembly] Cleanup toolchain test files. NFC.

2019-06-10 Thread Sam Clegg via cfe-commits
Author: sbc Date: Mon Jun 10 09:18:04 2019 New Revision: 362959 URL: http://llvm.org/viewvc/llvm-project?rev=362959=rev Log: [WebAssembly] Cleanup toolchain test files. NFC. Summary: Split up long lines to improve test readability. Subscribers: dschuff, jgravelle-google, aheejin, sunfish, jfb,

r362180 - Fix -DBUILD_SHARED_LIBS=ON build after rL362160

2019-05-30 Thread Sam Clegg via cfe-commits
Author: sbc Date: Thu May 30 18:04:00 2019 New Revision: 362180 URL: http://llvm.org/viewvc/llvm-project?rev=362180=rev Log: Fix -DBUILD_SHARED_LIBS=ON build after rL362160 Differential Revision: https://reviews.llvm.org/D62709 Modified: cfe/trunk/lib/CodeGen/CMakeLists.txt Modified:

r361651 - [WebAssembly] Use "linker" as linker shortname.

2019-05-24 Thread Sam Clegg via cfe-commits
Author: sbc Date: Fri May 24 10:36:07 2019 New Revision: 361651 URL: http://llvm.org/viewvc/llvm-project?rev=361651=rev Log: [WebAssembly] Use "linker" as linker shortname. This is in line with other platforms. Also, move the single statement methods into the header (also in line with other

r357195 - [WebAssembly] Reland of rL356953 (4dcf3acce6)

2019-03-28 Thread Sam Clegg via cfe-commits
Author: sbc Date: Thu Mar 28 10:45:18 2019 New Revision: 357195 URL: http://llvm.org/viewvc/llvm-project?rev=357195=rev Log: [WebAssembly] Reland of rL356953 (4dcf3acce6) The previous patch was missing GetProgramPath() in the return value of getLinkerPath(). See https://reviews.llvm.org/D59743

r356953 - [WebAssembly] Don't use default GetLinkerPath

2019-03-25 Thread Sam Clegg via cfe-commits
Author: sbc Date: Mon Mar 25 14:14:26 2019 New Revision: 356953 URL: http://llvm.org/viewvc/llvm-project?rev=356953=rev Log: [WebAssembly] Don't use default GetLinkerPath We can't (don't want to) honor the same set of "-fuse-ld" flags with WebAssembly since the ELF linkers (ld.lld, ld.gnu, etc)

r356847 - [WebAssembly] Fix test/Driver/wasm-toolchain.c in the presence of CLANG_DEFAULT_LINKER

2019-03-23 Thread Sam Clegg via cfe-commits
Author: sbc Date: Sat Mar 23 17:03:41 2019 New Revision: 356847 URL: http://llvm.org/viewvc/llvm-project?rev=356847=rev Log: [WebAssembly] Fix test/Driver/wasm-toolchain.c in the presence of CLANG_DEFAULT_LINKER This was broken in rL356817 (See https://reviews.llvm.org/D59721) Modified:

Re: r339163 - [WebAssembly] Remove use of lld -flavor flag

2018-10-17 Thread Sam Clegg via cfe-commits
quot;, but the wasm lld symlink is called > "wasm-ld". Does the driver need updating? Is the symlink name wrong? Am I > doing something wrong? > > On Tue, Aug 7, 2018 at 2:56 PM Sam Clegg via cfe-commits > wrote: >> >> Author: sbc >> Date: Tue Aug 7 11:55:41 2

r339163 - [WebAssembly] Remove use of lld -flavor flag

2018-08-07 Thread Sam Clegg via cfe-commits
Author: sbc Date: Tue Aug 7 11:55:41 2018 New Revision: 339163 URL: http://llvm.org/viewvc/llvm-project?rev=339163=rev Log: [WebAssembly] Remove use of lld -flavor flag This flag is deprecated. The preferred way to select the lld flavor is by calling it by one of its aliases. Differential

[libcxxabi] r337982 - [CMake] Don't use LIBCXXABI_ENABLE_STATIC option before its declared

2018-07-25 Thread Sam Clegg via cfe-commits
Author: sbc Date: Wed Jul 25 16:13:00 2018 New Revision: 337982 URL: http://llvm.org/viewvc/llvm-project?rev=337982=rev Log: [CMake] Don't use LIBCXXABI_ENABLE_STATIC option before its declared Summary: rL337867 introduced two new cmake_dependent_option options: -

r335510 - [WebAssembly] Add no-prototype attribute to prototype-less C functions

2018-06-25 Thread Sam Clegg via cfe-commits
Author: sbc Date: Mon Jun 25 11:47:32 2018 New Revision: 335510 URL: http://llvm.org/viewvc/llvm-project?rev=335510=rev Log: [WebAssembly] Add no-prototype attribute to prototype-less C functions The WebAssembly backend in particular benefits from being able to distinguish between varargs

r335380 - Fix BUILD_SHARED_LIBS=1 build of libclangHandleLLVM

2018-06-22 Thread Sam Clegg via cfe-commits
Author: sbc Date: Fri Jun 22 12:44:48 2018 New Revision: 335380 URL: http://llvm.org/viewvc/llvm-project?rev=335380=rev Log: Fix BUILD_SHARED_LIBS=1 build of libclangHandleLLVM Differential Revision: https://reviews.llvm.org/D48503 Modified:

r329323 - Fix typo in comment -fmath-errno=0 -> -fno-math-errno

2018-04-05 Thread Sam Clegg via cfe-commits
Author: sbc Date: Thu Apr 5 10:44:08 2018 New Revision: 329323 URL: http://llvm.org/viewvc/llvm-project?rev=329323=rev Log: Fix typo in comment -fmath-errno=0 -> -fno-math-errno The former is not a valid clang argument Differential Revision: https://reviews.llvm.org/D45102 Modified:

r323897 - [WebAssembly] Don't pass -ffunction-section/-fdata-sections

2018-01-31 Thread Sam Clegg via cfe-commits
Author: sbc Date: Wed Jan 31 10:55:22 2018 New Revision: 323897 URL: http://llvm.org/viewvc/llvm-project?rev=323897=rev Log: [WebAssembly] Don't pass -ffunction-section/-fdata-sections llvm currently forces both of these to true to passing them is redundant. Differential Revision:

r322880 - Convert comment to C-style to prevent warning

2018-01-18 Thread Sam Clegg via cfe-commits
Author: sbc Date: Thu Jan 18 11:31:33 2018 New Revision: 322880 URL: http://llvm.org/viewvc/llvm-project?rev=322880=rev Log: Convert comment to C-style to prevent warning Modified: cfe/trunk/tools/c-index-test/c-index-test.c Modified: cfe/trunk/tools/c-index-test/c-index-test.c URL:

r322382 - [WebAssembly] Support -stdlib=libc++ switch

2018-01-12 Thread Sam Clegg via cfe-commits
Author: sbc Date: Fri Jan 12 09:54:49 2018 New Revision: 322382 URL: http://llvm.org/viewvc/llvm-project?rev=322382=rev Log: [WebAssembly] Support -stdlib=libc++ switch Referenced implementation from Fuchsia and Darwin Toolchain. Still only support CST_Libcxx. Now checks that the argument is

r322321 - [WebAssembly] Remove `-allow-undefined-file wasm.syms` from linker args

2018-01-11 Thread Sam Clegg via cfe-commits
Author: sbc Date: Thu Jan 11 14:33:50 2018 New Revision: 322321 URL: http://llvm.org/viewvc/llvm-project?rev=322321=rev Log: [WebAssembly] Remove `-allow-undefined-file wasm.syms` from linker args See: https://github.com/WebAssembly/tool-conventions/issues/35 Differential Revision:

[libcxx] r320925 - [libcxx] Add WebAssembly support

2017-12-16 Thread Sam Clegg via cfe-commits
Author: sbc Date: Sat Dec 16 10:59:50 2017 New Revision: 320925 URL: http://llvm.org/viewvc/llvm-project?rev=320925=rev Log: [libcxx] Add WebAssembly support It turns out that this is the only change required in libcxx for it to compile with the new `wasm32-unknown-unknown-wasm` target recently

r319623 - [WebAssembly] Pass through --undefined to Wasm LLD

2017-12-02 Thread Sam Clegg via cfe-commits
Author: sbc Date: Sat Dec 2 15:11:13 2017 New Revision: 319623 URL: http://llvm.org/viewvc/llvm-project?rev=319623=rev Log: [WebAssembly] Pass through --undefined to Wasm LLD This is a follow-on to D40724 (Wasm entrypoint changes #1, add `--undefined` argument to LLD). Patch by Nicholas Wilson

r316777 - [WebAssembly] Add crt1.o with calling lld

2017-10-27 Thread Sam Clegg via cfe-commits
Author: sbc Date: Fri Oct 27 11:10:19 2017 New Revision: 316777 URL: http://llvm.org/viewvc/llvm-project?rev=316777=rev Log: [WebAssembly] Add crt1.o with calling lld Also, for OS unknown targets like wasm, don't include 'unknown' in the library path. This is a fix for rL316719. Differential

r316725 - Fix test/Driver/wasm-toolchain.c on windows

2017-10-26 Thread Sam Clegg via cfe-commits
Author: sbc Date: Thu Oct 26 18:20:16 2017 New Revision: 316725 URL: http://llvm.org/viewvc/llvm-project?rev=316725=rev Log: Fix test/Driver/wasm-toolchain.c on windows Modified: cfe/trunk/test/Driver/wasm-toolchain.c Modified: cfe/trunk/test/Driver/wasm-toolchain.c URL:

r313299 - [WebAssembly] Remove invliad lld arguments

2017-09-15 Thread Sam Clegg via cfe-commits
Author: sbc Date: Thu Sep 14 14:29:25 2017 New Revision: 313299 URL: http://llvm.org/viewvc/llvm-project?rev=313299=rev Log: [WebAssembly] Remove invliad lld arguments These arguments don't (not yet at least) make sense for the wasm lld port. Subscribers: jfb, dschuff, jgravelle-google, aheejin

r313307 - [WebAssembly] Fix wasm-toolchain.c tests

2017-09-15 Thread Sam Clegg via cfe-commits
Author: sbc Date: Thu Sep 14 15:36:44 2017 New Revision: 313307 URL: http://llvm.org/viewvc/llvm-project?rev=313307=rev Log: [WebAssembly] Fix wasm-toolchain.c tests Summary: This test should have been updated in r313299 Subscribers: jfb, dschuff, jgravelle-google, aheejin Differential

r306074 - [WebAssembly] Add default -allow-undefined-file to linker args

2017-06-23 Thread Sam Clegg via cfe-commits
Author: sbc Date: Thu Jun 22 19:02:55 2017 New Revision: 306074 URL: http://llvm.org/viewvc/llvm-project?rev=306074=rev Log: [WebAssembly] Add default -allow-undefined-file to linker args Also, don't use the outdated lib32/lib64 naming of files within the sysroot. The more modern/flexible

r302569 - [WebAssembly] Fix failing wasm-toolchain test

2017-05-09 Thread Sam Clegg via cfe-commits
Author: sbc Date: Tue May 9 13:44:23 2017 New Revision: 302569 URL: http://llvm.org/viewvc/llvm-project?rev=302569=rev Log: [WebAssembly] Fix failing wasm-toolchain test This test was broken in r302558. Differential Revision: https://reviews.llvm.org/D33015 Modified:

r302558 - [WebAssembly] Fix location and -flavor when running lld

2017-05-09 Thread Sam Clegg via cfe-commits
Author: sbc Date: Tue May 9 12:47:50 2017 New Revision: 302558 URL: http://llvm.org/viewvc/llvm-project?rev=302558=rev Log: [WebAssembly] Fix location and -flavor when running lld Add the toolchain installation directory to the program path so that lld can be found. Change -flavor to wasm.