malcolm.parsons added inline comments.

================
Comment at: lib/Sema/SemaLambda.cpp:1491
   else
     diag << From.getVariable();
   diag << From.isNonODRUsed();
----------------
dim wrote:
> Just for sanity's sake, I would still put an assert here, that 
> `getVariable()` does not return `nullptr`.  That might catch similar issues 
> in the future.  (And it is better than a segfault... :) )
> 
I'm adding an assert to `getVariable()` instead.


Repository:
  rC Clang

https://reviews.llvm.org/D41016



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

Reply via email to