[PATCH] D122215: [WebAssembly] Initial support for reference type externref in clang
Paulo Matos via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jan 16 05:49:23 PST 2023
pmatos added a comment.
I feel like the latest patch addresses all concerns until now. What do you think?
================
Comment at: clang/include/clang/Basic/WebAssemblyReferenceTypes.def:16
+//
+// - Name is the name of the builtin type. MangledName is the mangled name.
+//
----------------
aaron.ballman wrote:
> aaron.ballman wrote:
> > What kind of mangling is this name? Itanium? Microsoft? Something else?
> Still wondering about this.
This is just a generic name to base the actual mangling on tbh.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D122215/new/
https://reviews.llvm.org/D122215
More information about the cfe-commits
mailing list