[PATCH] D74116: [Sema][C++] Strawman patch to propagate conversion type in order to specialize the diagnostics

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 14 09:54:10 PST 2020


rjmccall added a comment.

The C diagnostic changes seem like improvements, yeah.  The virtue of re-using the same infrastructure across language modes.



================
Comment at: clang/lib/Sema/SemaExprCXX.cpp:3880
+        ToType, From->getType(), From, Action);
+    // assert(Diagnosed && "failed to diagnose bad conversion");
+    (void)Diagnosed;
----------------
This assertion seems important; can we make it work?


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

https://reviews.llvm.org/D74116





More information about the cfe-commits mailing list