nickdesaulniers added a comment. In D69876#1741294 <https://reviews.llvm.org/D69876#1741294>, @void wrote:
> In D69876#1740286 <https://reviews.llvm.org/D69876#1740286>, @jyknight wrote: > > > I think -Wuninitialized (UninitializedValues.cpp) should be taught how to > > detect the use of output variables in error blocks, at least for trivial > > cases. > > > > Actually, for some reason -- it looks like the warning is failing the wrong > > way right now, and emits an uninitialized use warning even where there > > shouldn't be one. > > > Nice catch! I updated the patch with a fix. PTAL. Please add a test for that! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69876/new/ https://reviews.llvm.org/D69876 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
