[PATCH] D87839: [SyntaxTree] Test the List API

Eduardo Caldas via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 22 02:56:33 PDT 2020


eduucaldas added a reviewer: gribozavr2.
eduucaldas added a comment.

I made a separate class for the tests on Lists, as it didn't share any methods with the tests for Trees. What do you think about that?

Should I also put the tests for lists in a different file, even though `TreeTest.cpp` could mean test `Tree.h`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87839



More information about the cfe-commits mailing list