erichkeane added inline comments.

================
Comment at: clang/test/SemaTemplate/concepts-lambda.cpp:26
+  template <typename T>
+  auto f2 = [](auto... args)
+    requires (sizeof...(args) > 0)
----------------
Huh... this f2 example seems to ICE for some reason, I could swear it worked, 
but building a different patch on top of this one showed me this fails.

Looking into it now, but it isn't critical to this patch, so if the rest looks 
good, I might remove this test case and fix it in a followup.


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

https://reviews.llvm.org/D134874

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D1348... Erich Keane via Phabricator via cfe-commits
    • [PATCH] ... Erich Keane via Phabricator via cfe-commits
    • [PATCH] ... Erich Keane via Phabricator via cfe-commits
    • [PATCH] ... Shafik Yaghmour via Phabricator via cfe-commits
    • [PATCH] ... Erich Keane via Phabricator via cfe-commits
    • [PATCH] ... Tom Honermann via Phabricator via cfe-commits
    • [PATCH] ... Erich Keane via Phabricator via cfe-commits
    • [PATCH] ... Erich Keane via Phabricator via cfe-commits
    • [PATCH] ... Johel Ernesto Guerrero Peña via Phabricator via cfe-commits
    • [PATCH] ... Tom Honermann via Phabricator via cfe-commits
    • [PATCH] ... Erich Keane via Phabricator via cfe-commits
    • [PATCH] ... Erich Keane via Phabricator via cfe-commits

Reply via email to