[PATCH] D24997: [ClangTidy] Add UsingInserter and NamespaceAliaser
Vassil Vassilev via cfe-commits
cfe-commits at lists.llvm.org
Wed Oct 19 06:31:25 PDT 2016
v.g.vassilev added inline comments.
================
Comment at: clang-tidy/utils/UsingInserter.cpp:58
+ if (AlreadyHasUsingDecl) {
+ AddedUsing.emplace(NameInFunction(Function, QualifiedName.str()));
+ return None;
----------------
Using emplace seems to break our modules libstdc++ 4.7 builds: http://lab.llvm.org:8011/builders/clang-x86_64-linux-selfhost-modules-2/builds/6066 . Doesn't that break other 4.7 or earlier builds?
https://reviews.llvm.org/D24997
More information about the cfe-commits
mailing list