diff options
author | Kirill Bobyrev <omtcyfz@gmail.com> | 2016-09-16 08:45:19 +0000 |
---|---|---|
committer | Kirill Bobyrev <omtcyfz@gmail.com> | 2016-09-16 08:45:19 +0000 |
commit | e5e7e153b5c7691b861b014844e760dccca42a16 (patch) | |
tree | 55b9afd6c1fbd30736653efbcc4d6f679cb166b9 /clang-tools-extra/test/clang-rename/ClassTestMulti.cpp | |
parent | b4e15b8ceeaf891a5ca5a8192f0719c5bda6d7ce (diff) | |
download | bcm5719-llvm-e5e7e153b5c7691b861b014844e760dccca42a16.tar.gz bcm5719-llvm-e5e7e153b5c7691b861b014844e760dccca42a16.zip |
[clang-rename] Merge rename-{at|all} & optimise.
Having both rename-at and rename-all both seems confusing and introduces
unneeded difficulties. After merging rename-at and rename-all maintaining main
function wrappers and custom help becomes redundant while CLI becomes less
confusing.
D24224 (which was the original patch causing buildbot failures) wasn't aware of
bugs caused by passing both -offset and -qualified-name. After D24224 was landed
it caused buildbot failures and therefor I just reverted it.
Two things that make this patch different from D24224 are:
* unittests/clang-rename was deleted, because it is unmaintained and doesn't do
much.
* Passing both `-offset` and `-qualified-name` isn't allowed anymore for the
sake of preventing bugs.
This patch is a trivial enhancement of accepted D24224 revision.
Tested with `ninja check-all`.
Differential Revision: https://reviews.llvm.org/D24567
llvm-svn: 281710
Diffstat (limited to 'clang-tools-extra/test/clang-rename/ClassTestMulti.cpp')
-rw-r--r-- | clang-tools-extra/test/clang-rename/ClassTestMulti.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/clang-tools-extra/test/clang-rename/ClassTestMulti.cpp b/clang-tools-extra/test/clang-rename/ClassTestMulti.cpp index 22911dfa58b..81e65c76065 100644 --- a/clang-tools-extra/test/clang-rename/ClassTestMulti.cpp +++ b/clang-tools-extra/test/clang-rename/ClassTestMulti.cpp @@ -5,7 +5,7 @@ class Foo2 /* Offset 2 */ { // CHECK: class Bar2 /* Offset 2 */ { }; // Test 1. -// RUN: clang-rename rename-all -offset=6 -new-name=Bar1 -offset=76 -new-name=Bar2 %s -- | sed 's,//.*,,' | FileCheck %s +// RUN: clang-rename -offset=6 -new-name=Bar1 -offset=76 -new-name=Bar2 %s -- | sed 's,//.*,,' | FileCheck %s // To find offsets after modifying the file, use: // grep -Ubo 'Foo.*' <file> |