[PATCH] D35701: Break up Targets.cpp into a header/impl pair per target type[NFCI]

Craig Topper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 21 10:21:55 PDT 2017


craig.topper added a comment.

I think we should drop "using namespace llvm;" from the cpp files. clang doesn't usually do that except in codegen and it doesn't look like it was required in the original Targets.cpp.


https://reviews.llvm.org/D35701





More information about the cfe-commits mailing list