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

[CJ DiMeglio via Phabricator via cfe-commits]

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