erichkeane added a comment.

I agree with Corentin here, I've got no idea what the RIGHT solution looks like 
here, but this looks to be at least an improvement.  1 nit, plus  need a 
release note.  Otherwise LGTM.



================
Comment at: clang/test/CodeGenCXX/cxx20-unevaluated-lambda-crash.cpp:4
+// CHECK-LABEL: define linkonce_odr void 
@"_ZN10Issue579601EIiEENS_1FILNS_3$_0EEEEv"()
+namespace Issue57960{
+template<auto>
----------------
Nit: need a space after the name and before the {.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140554

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

Reply via email to