[PATCH] D59196: [NFC][clang][PCH] ASTStmtReader::VisitStmt(): fixup faulty assert.

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 15 14:27:07 PDT 2019


lebedev.ri added a reviewer: dexonsmith.
lebedev.ri added a comment.

In D59196#1429393 <https://reviews.llvm.org/D59196#1429393>, @riccibruno wrote:

> I am not an expert in the serialization code (just did some modifications), but this seems reasonable to me.


That's my thoughts too..
Unless of course it is also testing that `NumStmtFields` is zero.
Anyone with more knowledged opinion?


Repository:
  rC Clang

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

https://reviews.llvm.org/D59196





More information about the cfe-commits mailing list