maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity
328,070 Commits 23 Branches 0 Tags 2.2 GiB
Commit Graph

11 Commits

Author SHA1 Message Date
Haojian Wu 3626516b14 [clang-move] Only move used helper declarations. 
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
 2017-01-03 09:00:51 +00:00
Haojian Wu 50a45d9371 [clang-move] Fix not moving using-decls in global namespace in old.cc 
Reviewers: ioeric

Subscribers: cfe-commits

Differential Revision: https://reviews.llvm.org/D26844

llvm-svn: 287330
 2016-11-18 10:51:16 +00:00
Haojian Wu 67bb651ad1 [clang-move] Move using-decl in old cc. 
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
 2016-10-19 14:13:21 +00:00
Haojian Wu 220c755dfc [clang-move] Add header guard for the new header. 
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
 2016-10-14 13:01:36 +00:00
Haojian Wu e77bcc7371 [clang-move] Better support enclosing class. 
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
 2016-10-13 10:31:00 +00:00
Haojian Wu 9df3ac1f56 Recommit r283538 "[clang-move] Support moving multiple classes in one run." 
llvm-svn: 284109
 2016-10-13 08:48:42 +00:00
Renato Golin 0dccd1a0fd Revert "[clang-move] Support moving multiple classes in one run." 
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
 2016-10-07 13:58:10 +00:00
Haojian Wu 1854bd5ae8 [clang-move] Simplify lint tests 
No need to use compilation database.

llvm-svn: 283545
 2016-10-07 12:35:37 +00:00
Haojian Wu 91c3aa22a9 Revert "fix buildbot error" since it is not right fix. 
llvm-svn: 283538
 2016-10-07 10:59:03 +00:00
Haojian Wu 18424c8e5a Fix buildbot error. 
The error maybe caused by the mixed environment of the two lint tests.
Cleanup the environment before running each test.

llvm-svn: 283534
 2016-10-07 09:23:28 +00:00
Haojian Wu a709e384ce [clang-move] Support moving multiple classes in one run. 
Reviewers: ioeric

Subscribers: cfe-commits

Differential Revision: https://reviews.llvm.org/D25309

llvm-svn: 283526
 2016-10-07 08:29:20 +00:00