diff options
| author | Daniel Jasper <djasper@google.com> | 2014-11-25 10:45:23 +0000 |
|---|---|---|
| committer | Daniel Jasper <djasper@google.com> | 2014-11-25 10:45:23 +0000 |
| commit | 4525be4ebc3f8d45daf26affb8960b5d5bc932bf (patch) | |
| tree | 64516f1e1d7a2ae698a82c30597a615ed14edba0 /clang-tools-extra/clang-tidy/misc/UseOverride.cpp | |
| parent | 1cbac4e94f4a667a94bcbe57adc535e0e390430d (diff) | |
| download | bcm5719-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.cpp | 6 |
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; } } } |

