diff options
author | Dmitri Gribenko <gribozavr@gmail.com> | 2020-05-29 14:12:51 +0200 |
---|---|---|
committer | Dmitri Gribenko <gribozavr@gmail.com> | 2020-05-29 14:25:20 +0200 |
commit | d4ef654673a921878ba5aedb9725b2ac32681f01 (patch) | |
tree | 2d2f231aff3a90c966b24d0581e76f2e7e035e34 /llvm/lib/CodeGen/ModuloSchedule.cpp | |
parent | ab4d02cf265982d4c03123d2f52b9d5ee8df575d (diff) | |
download | llvm-d4ef654673a921878ba5aedb9725b2ac32681f01.zip llvm-d4ef654673a921878ba5aedb9725b2ac32681f01.tar.gz llvm-d4ef654673a921878ba5aedb9725b2ac32681f01.tar.bz2 |
Rename APIs in unittests/AST/Language.h in preparation to share them
Summary:
Declaring these helpers in the ast_matcher namespace in the clangAST
unit test seems inappropriate -- neither these helpers, nor clangAST have
anything to do with AST matchers. Therefore, I moved these helpers to
the clang namespace.
Declaring another typedef called "ArgVector" is not a good idea -- we
already have both "ArgVector", "ArgsVector", and "ArgList". I expanded
it into the underlying type.
Declaring another enum called "Language" is not a good idea because we
arleady have the "clang::Language" enum. I renamed it to
"TestLanguage".
Similarly, I renamed "getBasicRunOptionsForLanguage" to
"getCommandLineArgsForTesting" to explain the semantics better (what are
"run options"?) and not repeat types in the function name
("ForLanguage").
Reviewers: shafik, rengolin, sammccall
Reviewed By: sammccall
Subscribers: gribozavr2, sammccall, martong, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D80786
Diffstat (limited to 'llvm/lib/CodeGen/ModuloSchedule.cpp')
0 files changed, 0 insertions, 0 deletions