[PATCH] D69876: Allow output constraints on "asm goto"

Mon Nov 11 14:29:07 PST 2019

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