cor3ntin added a comment.

@aaron.ballman I tried to address your comment, except that:

- There is a bug with default parameter in blocks so that can't be tested
- Clang does not support default initialization of capture,  and if it did, I 
don't thee how we could support immediate calls in that easily.
- Compound statements in parameters are not supported.



================
Comment at: clang/include/clang/Sema/Sema.h:1349-1350
+      DeclContext *Context = nullptr;
+
+      bool isValid() const { return Decl != nullptr; }
+    };
----------------
aaron.ballman wrote:
> This can now be removed as it's unused.
Nice catch!


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D136554/new/

https://reviews.llvm.org/D136554

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to