lethalantidote added inline comments.
================ Comment at: clang/lib/Sema/AnalysisBasedWarnings.cpp:907 else DiagUninitUse(S, VD, Use, true); } ---------------- thakis wrote: > Should the check be in DiagUninitUse()? Or is there a reason this one should > happen for volatiles? Good point. Thanks for catching that. I hoped I understood this use case correctly, lemme know if the update makes sense. https://reviews.llvm.org/D28543 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits