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

Bill Wendling via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 18 14:19:30 PST 2019


void marked 2 inline comments as done.
void added inline comments.


================
Comment at: clang/lib/Analysis/UninitializedValues.cpp:831
+  for (auto i = as->begin_outputs(), e = as->end_outputs(); i != e; ++i)
+    if (const VarDecl *VD = findVar(*i).getDecl())
+      vals[VD] = Initialized;
----------------
nickdesaulniers wrote:
> this is still not a range based for loop.
> 
> Does:
> 
> ```
> for (const auto &O : as->outputs())
>   if (const VarDecl *VD = findVar(O).getDecl())
>     vals[VD] = Initialized;
> ```
> not work?
Done.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69876/new/

https://reviews.llvm.org/D69876





More information about the cfe-commits mailing list