[PATCH] D64678: [Sema] Fix -Wuninitialized for struct assignment from GNU C statement expression

Nathan Huckleberry via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 15 10:40:45 PDT 2019


Nathan-Huckleberry marked 3 inline comments as done.
Nathan-Huckleberry added inline comments.


================
Comment at: clang/test/Sema/warn-uninitialized-statement-expression.c:21
+    struct widget z = my_widget; // expected-warning{{variable 'my_widget' is uninitialized when used within its own initialization}}
+    int x = my_widget.x;
+    init2(&my_widget);
----------------
nickdesaulniers wrote:
> This needs a trailing comment like:
> ```
> int x = my_widget.x; // fixme: we should probably warn about this case
> ```
> and file a bug about it.  Doesn't need to be solved here.
https://bugs.llvm.org/show_bug.cgi?id=42625


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D64678





More information about the cfe-commits mailing list