summaryrefslogtreecommitdiffstats
path: root/clang-tools-extra/test/clang-move/move-multiple-classes.cpp
Commit message (Collapse)AuthorAgeFilesLines
* [clang-move] Only move used helper declarations.Haojian Wu2017-01-031-6/+5
| | | | | | | | | | | | | | | | | Summary: Instead of moving all the helper declarations blindly, this patch implements an AST-based call graph solution to make clang-move only move used helper decls to new.cc and remove unused decls in old.cc. Depends on D27674. Reviewers: ioeric Subscribers: mgorny, cfe-commits Differential Revision: https://reviews.llvm.org/D27673 llvm-svn: 290873
* [clang-move] Fix not moving using-decls in global namespace in old.ccHaojian Wu2016-11-181-2/+10
| | | | | | | | | | Reviewers: ioeric Subscribers: cfe-commits Differential Revision: https://reviews.llvm.org/D26844 llvm-svn: 287330
* [clang-move] Move using-decl in old cc.Haojian Wu2016-10-191-1/+24
| | | | | | | | | | | | | | Summary: Another fix is to move the whole anonymous namespace declaration completely instead of moving fun/var declarations only. Reviewers: ioeric Subscribers: cfe-commits Differential Revision: https://reviews.llvm.org/D25762 llvm-svn: 284592
* [clang-move] Add header guard for the new header.Haojian Wu2016-10-141-0/+3
| | | | | | | | | | | | | | | | | | Summary: The header guard generated by clang-move isn't always a perfect style, just avoid getting the header included multiple times during compiling period. Also, we can use llvm-Header-guard clang-tidy check to correct the guard automatically. Reviewers: ioeric Subscribers: cfe-commits Differential Revision: https://reviews.llvm.org/D25610 llvm-svn: 284233
* [clang-move] Better support enclosing class.Haojian Wu2016-10-131-1/+15
| | | | | | | | | | | | | | | Summary: * When moving an outermost enclosing class, all its nested classes should also be moved together. * Add a test for not moving nested class. Reviewers: ioeric Subscribers: cfe-commits Differential Revision: https://reviews.llvm.org/D25369 llvm-svn: 284111
* Recommit r283538 "[clang-move] Support moving multiple classes in one run."Haojian Wu2016-10-131-0/+57
| | | | llvm-svn: 284109
* Revert "[clang-move] Support moving multiple classes in one run."Renato Golin2016-10-071-57/+0
| | | | | | | | | | | This reverts commit r283526 et al as it keeps randomly breaking bots, even after the commit has gone, on other people's commit ranges. Revert "[clang-move] Simplify lint tests" (r283545). Revert "Fix buildbot error." (r283534). Revert "Revert "fix buildbot error" since it is not right fix." (r283538). llvm-svn: 283553
* [clang-move] Simplify lint testsHaojian Wu2016-10-071-3/+2
| | | | | | No need to use compilation database. llvm-svn: 283545
* Revert "fix buildbot error" since it is not right fix.Haojian Wu2016-10-071-1/+0
| | | | llvm-svn: 283538
* Fix buildbot error.Haojian Wu2016-10-071-0/+1
| | | | | | | The error maybe caused by the mixed environment of the two lint tests. Cleanup the environment before running each test. llvm-svn: 283534
* [clang-move] Support moving multiple classes in one run.Haojian Wu2016-10-071-0/+58
Reviewers: ioeric Subscribers: cfe-commits Differential Revision: https://reviews.llvm.org/D25309 llvm-svn: 283526
OpenPOWER on IntegriCloud