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