[PATCH] D146194: NOT report the warning that unnamed bitfields are uninitialized.

Balázs Benics via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 16 06:04:12 PDT 2023


steakhal added a comment.

BTW we request full context for git patches. That way the reviewer can scroll and see the context around your hunks.
You can request git to emit the full context by passing the `-U999999` commandline option. See <https://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interface>.

As per your proposed change, I think we should do something like this:

  for (const auto *I : RD->fields()) {
    if (I->isUnnamedBitfield())
      continue;

instead of embedding this into some condition.

But I'm curious to see why does C behave differently.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146194



More information about the cfe-commits mailing list