[PATCH] D28543: Elliminates uninitialized warning for volitile varibles.

CJ DiMeglio via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 11 13:26:57 PST 2017

lethalantidote added inline comments.

Comment at: clang/lib/Sema/AnalysisBasedWarnings.cpp:907
       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. 


More information about the cfe-commits mailing list