Page MenuHomePhabricator

[SyntaxTree][List] `assertInvariants` for `List`s
ClosedPublic

Authored by eduucaldas on Mon, Sep 14, 1:23 AM.

Diff Detail

Event Timeline

eduucaldas created this revision.Mon, Sep 14, 1:23 AM
Herald added a project: Restricted Project. · View Herald TranscriptMon, Sep 14, 1:23 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
eduucaldas requested review of this revision.Mon, Sep 14, 1:23 AM
gribozavr2 accepted this revision.Mon, Sep 14, 1:53 AM
gribozavr2 added inline comments.
clang/lib/Tooling/Syntax/Tree.cpp
236
241

This assertion is true only for trees that represent correct code. To cover all cases, add the unknown role.

This revision is now accepted and ready to land.Mon, Sep 14, 1:53 AM
eduucaldas marked 2 inline comments as done.

Fix build error and clang-tidy lint error

This revision was landed with ongoing or failed builds.Mon, Sep 14, 3:45 AM
This revision was automatically updated to reflect the committed changes.