aaron.ballman added a comment.
Aside from the suggested changes, the Clang bits LGTM
================
Comment at: clang/lib/Sema/SemaDeclAttr.cpp:7636-7639
+static void handleWebAssemblyAsyncAttr(Sema &S, Decl *D, const ParsedAttr &AL)
{
+ D->addAttr(::new (S.Context) WebAssemblyAsyncAttr(S.Context, AL));
+}
+
----------------
This can be removed entirely (see edit below).
================
Comment at: clang/lib/Sema/SemaDeclAttr.cpp:8747
+ case ParsedAttr::AT_WebAssemblyAsync:
+ handleWebAssemblyAsyncAttr(S, D, AL);
+ break;
----------------
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D150803/new/
https://reviews.llvm.org/D150803
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
