summaryrefslogtreecommitdiffstats
path: root/clang-tools-extra/clang-tidy/misc/UseOverride.cpp
diff options
context:
space:
mode:
authorDaniel Jasper <djasper@google.com>2014-11-25 10:45:23 +0000
committerDaniel Jasper <djasper@google.com>2014-11-25 10:45:23 +0000
commit4525be4ebc3f8d45daf26affb8960b5d5bc932bf (patch)
tree64516f1e1d7a2ae698a82c30597a615ed14edba0 /clang-tools-extra/clang-tidy/misc/UseOverride.cpp
parent1cbac4e94f4a667a94bcbe57adc535e0e390430d (diff)
downloadbcm5719-llvm-4525be4ebc3f8d45daf26affb8960b5d5bc932bf.tar.gz
bcm5719-llvm-4525be4ebc3f8d45daf26affb8960b5d5bc932bf.zip
clang-tidy: Add override before the first attribute.
Apparently attributes aren't sorted by source location. llvm-svn: 222751
Diffstat (limited to 'clang-tools-extra/clang-tidy/misc/UseOverride.cpp')
-rw-r--r--clang-tools-extra/clang-tidy/misc/UseOverride.cpp6
1 files changed, 4 insertions, 2 deletions
diff --git a/clang-tools-extra/clang-tidy/misc/UseOverride.cpp b/clang-tools-extra/clang-tidy/misc/UseOverride.cpp
index bd55f894af8..2b9ef5bc681 100644
--- a/clang-tools-extra/clang-tidy/misc/UseOverride.cpp
+++ b/clang-tools-extra/clang-tidy/misc/UseOverride.cpp
@@ -98,8 +98,10 @@ void UseOverride::check(const MatchFinder::MatchResult &Result) {
if (Method->hasAttrs()) {
for (const clang::Attr *A : Method->getAttrs()) {
if (!A->isImplicit()) {
- InsertLoc = Sources.getExpansionLoc(A->getLocation());
- break;
+ SourceLocation Loc = Sources.getExpansionLoc(A->getLocation());
+ if (!InsertLoc.isValid() ||
+ Sources.isBeforeInTranslationUnit(Loc, InsertLoc))
+ InsertLoc = Loc;
}
}
}
OpenPOWER on IntegriCloud