[PATCH] D57659: [Sema] SequenceChecker: Add some comments + related small NFCs in preparation of the following patches

Bruno Ricci via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 4 07:18:46 PST 2019


riccibruno marked an inline comment as done.
riccibruno added inline comments.


================
Comment at: lib/Sema/SemaChecking.cpp:11748
+    /// The expressions corresponding to each usage kind.
+    Expr *UsageExprs[UK_Count]{};
+
----------------
aaron.ballman wrote:
> You can drop the `{}` here and below.
I don't think I can (although I can drop it for the array of `SequenceTree::Seq`). I want to value-initialize each expression in this array so that they are all null.


Repository:
  rC Clang

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

https://reviews.llvm.org/D57659





More information about the cfe-commits mailing list