[PATCH] D71823: Support Swift calling convention for WebAssembly targets

2020-01-24 Thread Derek Schuff via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGc5bd3d07262f: Support Swift calling convention for WebAssembly targets (authored by kateinoigakukun, committed by dschuff). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D71823: Support Swift calling convention for WebAssembly targets

2020-01-24 Thread Yuta Saito via Phabricator via cfe-commits
kateinoigakukun added a comment. Could you merge this? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71823/new/ https://reviews.llvm.org/D71823 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D71823: Support Swift calling convention for WebAssembly targets

2020-01-22 Thread Yuta Saito via Phabricator via cfe-commits
kateinoigakukun added a comment. In D71823#1834488 , @dschuff wrote: > Sorry we left this dangling for so long. This little change looks fine; I > guess you must be in the process of porting swift to wasm? Do you have any > more detailed plans for that

[PATCH] D71823: Support Swift calling convention for WebAssembly targets

2020-01-22 Thread Derek Schuff via Phabricator via cfe-commits
dschuff accepted this revision. dschuff added a comment. This revision is now accepted and ready to land. Sorry we left this dangling for so long. This little change looks fine; I guess you must be in the process of porting swift to wasm? Do you have any more detailed plans for that written

[PATCH] D71823: Support Swift calling convention for WebAssembly targets

2020-01-11 Thread Yuta Saito via Phabricator via cfe-commits
kateinoigakukun marked an inline comment as done. kateinoigakukun added inline comments. Comment at: clang/lib/Basic/Targets/WebAssembly.h:122 +case CC_Swift: + return CCCR_OK; +default: sbc100 wrote: > This seems to disagree with the longer list in

[PATCH] D71823: Support Swift calling convention for WebAssembly targets

2020-01-11 Thread Sam Clegg via Phabricator via cfe-commits
sbc100 added inline comments. Comment at: clang/lib/Basic/Targets/WebAssembly.h:122 +case CC_Swift: + return CCCR_OK; +default: This seems to disagree with the longer list in `callingConvSupported`.Is this going to generate unnecessary

[PATCH] D71823: Support Swift calling convention for WebAssembly targets

2019-12-22 Thread Yuta Saito via Phabricator via cfe-commits
kateinoigakukun updated this revision to Diff 235085. Herald added subscribers: llvm-commits, hiraditya. Herald added a project: LLVM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71823/new/ https://reviews.llvm.org/D71823 Files:

[PATCH] D71823: Support Swift calling convention for WebAssembly targets

2019-12-22 Thread Yuta Saito via Phabricator via cfe-commits
kateinoigakukun created this revision. Herald added subscribers: cfe-commits, sunfish, aheejin, jgravelle-google, sbc100, dschuff. Herald added a project: clang. Support Swift calling convention for WebAssembly targets This adds basic support for the Swift calling convention with WebAssembly