[PATCH] D114721: [clang][dataflow] Add unit tests for PostOrderCFGView
Yitzhak Mandelbaum via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Nov 29 10:10:27 PST 2021
ymandel added a comment.
Looks good, but why split the test into its own directory? I see that the implementation file is in clang/lib/Analysis and, in general, the lib and unittest directories are often flatter than the corresponding include directories. Maybe just put it directly into unittests/Analysis?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D114721/new/
https://reviews.llvm.org/D114721
More information about the cfe-commits
mailing list