dim accepted this revision.
dim added a comment.
This revision is now accepted and ready to land.
For the rest, LGTM. It fixes the segfault, for both the full original test
case, and my reduced version.
================
Comment at: lib/Sema/SemaLambda.cpp:1491
else
diag << From.getVariable();
diag << From.isNonODRUsed();
----------------
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... :) )
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
