[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);
> 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