[PATCH] D61838: [Sema] Suppress additional warnings for C's zero initializer

Peter Wu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 15 15:56:06 PDT 2019


Lekensteyn added a comment.

@al3xtjames I was about to commit this but noticed that some others check whether `getInit(0)` is NULL or not before proceeding. Should that be done here as well? If not, why?

See for example "Harden InitListExpr::isStringLiteralInit() against getInit() returning null." in
https://github.com/llvm/llvm-project/commit/256bd96d1c2464b0f0ecb482ca52075a3158f6a1#diff-dac09655ff6a54658c320a28a6ea297c
and InitListExpr::isTransparent in
https://github.com/llvm/llvm-project/commit/122f88d481971b68d05ba12395c3b73ab4b31ab3#diff-dac09655ff6a54658c320a28a6ea297c


Repository:
  rC Clang

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

https://reviews.llvm.org/D61838





More information about the llvm-commits mailing list