1349cc55cSDimitry Andric //===--- LeftRightQualifierAlignmentFixer.cpp -------------------*- C++--*-===// 2349cc55cSDimitry Andric // 3349cc55cSDimitry Andric // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. 4349cc55cSDimitry Andric // See https://llvm.org/LICENSE.txt for license information. 5349cc55cSDimitry Andric // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception 6349cc55cSDimitry Andric // 7349cc55cSDimitry Andric //===----------------------------------------------------------------------===// 8349cc55cSDimitry Andric /// 9349cc55cSDimitry Andric /// \file 10349cc55cSDimitry Andric /// This file implements LeftRightQualifierAlignmentFixer, a TokenAnalyzer that 11349cc55cSDimitry Andric /// enforces either left or right const depending on the style. 12349cc55cSDimitry Andric /// 13349cc55cSDimitry Andric //===----------------------------------------------------------------------===// 14349cc55cSDimitry Andric 15349cc55cSDimitry Andric #include "QualifierAlignmentFixer.h" 16349cc55cSDimitry Andric #include "FormatToken.h" 17349cc55cSDimitry Andric #include "llvm/Support/Debug.h" 18349cc55cSDimitry Andric #include "llvm/Support/Regex.h" 19349cc55cSDimitry Andric 20349cc55cSDimitry Andric #include <algorithm> 21bdd1243dSDimitry Andric #include <optional> 22349cc55cSDimitry Andric 23349cc55cSDimitry Andric #define DEBUG_TYPE "format-qualifier-alignment-fixer" 24349cc55cSDimitry Andric 25349cc55cSDimitry Andric namespace clang { 26349cc55cSDimitry Andric namespace format { 27349cc55cSDimitry Andric 28349cc55cSDimitry Andric QualifierAlignmentFixer::QualifierAlignmentFixer( 29349cc55cSDimitry Andric const Environment &Env, const FormatStyle &Style, StringRef &Code, 30349cc55cSDimitry Andric ArrayRef<tooling::Range> Ranges, unsigned FirstStartColumn, 31349cc55cSDimitry Andric unsigned NextStartColumn, unsigned LastStartColumn, StringRef FileName) 32349cc55cSDimitry Andric : TokenAnalyzer(Env, Style), Code(Code), Ranges(Ranges), 33349cc55cSDimitry Andric FirstStartColumn(FirstStartColumn), NextStartColumn(NextStartColumn), 34349cc55cSDimitry Andric LastStartColumn(LastStartColumn), FileName(FileName) { 35349cc55cSDimitry Andric std::vector<std::string> LeftOrder; 36349cc55cSDimitry Andric std::vector<std::string> RightOrder; 37349cc55cSDimitry Andric std::vector<tok::TokenKind> ConfiguredQualifierTokens; 38349cc55cSDimitry Andric PrepareLeftRightOrdering(Style.QualifierOrder, LeftOrder, RightOrder, 39349cc55cSDimitry Andric ConfiguredQualifierTokens); 40349cc55cSDimitry Andric 4181ad6265SDimitry Andric // Handle the left and right alignment separately. 42349cc55cSDimitry Andric for (const auto &Qualifier : LeftOrder) { 43349cc55cSDimitry Andric Passes.emplace_back( 44349cc55cSDimitry Andric [&, Qualifier, ConfiguredQualifierTokens](const Environment &Env) { 45349cc55cSDimitry Andric return LeftRightQualifierAlignmentFixer(Env, Style, Qualifier, 46349cc55cSDimitry Andric ConfiguredQualifierTokens, 47349cc55cSDimitry Andric /*RightAlign=*/false) 48349cc55cSDimitry Andric .process(); 49349cc55cSDimitry Andric }); 50349cc55cSDimitry Andric } 51349cc55cSDimitry Andric for (const auto &Qualifier : RightOrder) { 52349cc55cSDimitry Andric Passes.emplace_back( 53349cc55cSDimitry Andric [&, Qualifier, ConfiguredQualifierTokens](const Environment &Env) { 54349cc55cSDimitry Andric return LeftRightQualifierAlignmentFixer(Env, Style, Qualifier, 55349cc55cSDimitry Andric ConfiguredQualifierTokens, 56349cc55cSDimitry Andric /*RightAlign=*/true) 57349cc55cSDimitry Andric .process(); 58349cc55cSDimitry Andric }); 59349cc55cSDimitry Andric } 60349cc55cSDimitry Andric } 61349cc55cSDimitry Andric 62349cc55cSDimitry Andric std::pair<tooling::Replacements, unsigned> QualifierAlignmentFixer::analyze( 6381ad6265SDimitry Andric TokenAnnotator & /*Annotator*/, 6481ad6265SDimitry Andric SmallVectorImpl<AnnotatedLine *> & /*AnnotatedLines*/, 6581ad6265SDimitry Andric FormatTokenLexer & /*Tokens*/) { 66349cc55cSDimitry Andric auto Env = Environment::make(Code, FileName, Ranges, FirstStartColumn, 67349cc55cSDimitry Andric NextStartColumn, LastStartColumn); 68349cc55cSDimitry Andric if (!Env) 69349cc55cSDimitry Andric return {}; 70bdd1243dSDimitry Andric std::optional<std::string> CurrentCode; 71349cc55cSDimitry Andric tooling::Replacements Fixes; 72349cc55cSDimitry Andric for (size_t I = 0, E = Passes.size(); I < E; ++I) { 73349cc55cSDimitry Andric std::pair<tooling::Replacements, unsigned> PassFixes = Passes[I](*Env); 74349cc55cSDimitry Andric auto NewCode = applyAllReplacements( 75349cc55cSDimitry Andric CurrentCode ? StringRef(*CurrentCode) : Code, PassFixes.first); 76349cc55cSDimitry Andric if (NewCode) { 77349cc55cSDimitry Andric Fixes = Fixes.merge(PassFixes.first); 78349cc55cSDimitry Andric if (I + 1 < E) { 79349cc55cSDimitry Andric CurrentCode = std::move(*NewCode); 80349cc55cSDimitry Andric Env = Environment::make( 81349cc55cSDimitry Andric *CurrentCode, FileName, 82349cc55cSDimitry Andric tooling::calculateRangesAfterReplacements(Fixes, Ranges), 83349cc55cSDimitry Andric FirstStartColumn, NextStartColumn, LastStartColumn); 84349cc55cSDimitry Andric if (!Env) 85349cc55cSDimitry Andric return {}; 86349cc55cSDimitry Andric } 87349cc55cSDimitry Andric } 88349cc55cSDimitry Andric } 89349cc55cSDimitry Andric 90349cc55cSDimitry Andric // Don't make replacements that replace nothing. 91349cc55cSDimitry Andric tooling::Replacements NonNoOpFixes; 92349cc55cSDimitry Andric 9304eeddc0SDimitry Andric for (const tooling::Replacement &Fix : Fixes) { 9404eeddc0SDimitry Andric StringRef OriginalCode = Code.substr(Fix.getOffset(), Fix.getLength()); 95349cc55cSDimitry Andric 9604eeddc0SDimitry Andric if (!OriginalCode.equals(Fix.getReplacementText())) { 9704eeddc0SDimitry Andric auto Err = NonNoOpFixes.add(Fix); 9881ad6265SDimitry Andric if (Err) { 99349cc55cSDimitry Andric llvm::errs() << "Error adding replacements : " 100349cc55cSDimitry Andric << llvm::toString(std::move(Err)) << "\n"; 101349cc55cSDimitry Andric } 102349cc55cSDimitry Andric } 10381ad6265SDimitry Andric } 104349cc55cSDimitry Andric return {NonNoOpFixes, 0}; 105349cc55cSDimitry Andric } 106349cc55cSDimitry Andric 107349cc55cSDimitry Andric static void replaceToken(const SourceManager &SourceMgr, 108349cc55cSDimitry Andric tooling::Replacements &Fixes, 109349cc55cSDimitry Andric const CharSourceRange &Range, std::string NewText) { 110349cc55cSDimitry Andric auto Replacement = tooling::Replacement(SourceMgr, Range, NewText); 111349cc55cSDimitry Andric auto Err = Fixes.add(Replacement); 112349cc55cSDimitry Andric 11381ad6265SDimitry Andric if (Err) { 114349cc55cSDimitry Andric llvm::errs() << "Error while rearranging Qualifier : " 115349cc55cSDimitry Andric << llvm::toString(std::move(Err)) << "\n"; 116349cc55cSDimitry Andric } 11781ad6265SDimitry Andric } 118349cc55cSDimitry Andric 119349cc55cSDimitry Andric static void removeToken(const SourceManager &SourceMgr, 120349cc55cSDimitry Andric tooling::Replacements &Fixes, 121349cc55cSDimitry Andric const FormatToken *First) { 122349cc55cSDimitry Andric auto Range = CharSourceRange::getCharRange(First->getStartOfNonWhitespace(), 123349cc55cSDimitry Andric First->Tok.getEndLoc()); 124349cc55cSDimitry Andric replaceToken(SourceMgr, Fixes, Range, ""); 125349cc55cSDimitry Andric } 126349cc55cSDimitry Andric 127349cc55cSDimitry Andric static void insertQualifierAfter(const SourceManager &SourceMgr, 128349cc55cSDimitry Andric tooling::Replacements &Fixes, 129349cc55cSDimitry Andric const FormatToken *First, 130349cc55cSDimitry Andric const std::string &Qualifier) { 131349cc55cSDimitry Andric FormatToken *Next = First->Next; 132349cc55cSDimitry Andric if (!Next) 133349cc55cSDimitry Andric return; 134349cc55cSDimitry Andric auto Range = CharSourceRange::getCharRange(Next->getStartOfNonWhitespace(), 135349cc55cSDimitry Andric Next->Tok.getEndLoc()); 136349cc55cSDimitry Andric 137349cc55cSDimitry Andric std::string NewText = " " + Qualifier + " "; 138349cc55cSDimitry Andric NewText += Next->TokenText; 139349cc55cSDimitry Andric replaceToken(SourceMgr, Fixes, Range, NewText); 140349cc55cSDimitry Andric } 141349cc55cSDimitry Andric 142349cc55cSDimitry Andric static void insertQualifierBefore(const SourceManager &SourceMgr, 143349cc55cSDimitry Andric tooling::Replacements &Fixes, 144349cc55cSDimitry Andric const FormatToken *First, 145349cc55cSDimitry Andric const std::string &Qualifier) { 146349cc55cSDimitry Andric auto Range = CharSourceRange::getCharRange(First->getStartOfNonWhitespace(), 147349cc55cSDimitry Andric First->Tok.getEndLoc()); 148349cc55cSDimitry Andric 149349cc55cSDimitry Andric std::string NewText = " " + Qualifier + " "; 150349cc55cSDimitry Andric NewText += First->TokenText; 151349cc55cSDimitry Andric 152349cc55cSDimitry Andric replaceToken(SourceMgr, Fixes, Range, NewText); 153349cc55cSDimitry Andric } 154349cc55cSDimitry Andric 155349cc55cSDimitry Andric static bool endsWithSpace(const std::string &s) { 15681ad6265SDimitry Andric if (s.empty()) 157349cc55cSDimitry Andric return false; 158349cc55cSDimitry Andric return isspace(s.back()); 159349cc55cSDimitry Andric } 160349cc55cSDimitry Andric 161349cc55cSDimitry Andric static bool startsWithSpace(const std::string &s) { 16281ad6265SDimitry Andric if (s.empty()) 163349cc55cSDimitry Andric return false; 164349cc55cSDimitry Andric return isspace(s.front()); 165349cc55cSDimitry Andric } 166349cc55cSDimitry Andric 167349cc55cSDimitry Andric static void rotateTokens(const SourceManager &SourceMgr, 168349cc55cSDimitry Andric tooling::Replacements &Fixes, const FormatToken *First, 169349cc55cSDimitry Andric const FormatToken *Last, bool Left) { 170349cc55cSDimitry Andric auto *End = Last; 171349cc55cSDimitry Andric auto *Begin = First; 172349cc55cSDimitry Andric if (!Left) { 173349cc55cSDimitry Andric End = Last->Next; 174349cc55cSDimitry Andric Begin = First->Next; 175349cc55cSDimitry Andric } 176349cc55cSDimitry Andric 177349cc55cSDimitry Andric std::string NewText; 178349cc55cSDimitry Andric // If we are rotating to the left we move the Last token to the front. 179349cc55cSDimitry Andric if (Left) { 180349cc55cSDimitry Andric NewText += Last->TokenText; 181349cc55cSDimitry Andric NewText += " "; 182349cc55cSDimitry Andric } 183349cc55cSDimitry Andric 184349cc55cSDimitry Andric // Then move through the other tokens. 185349cc55cSDimitry Andric auto *Tok = Begin; 186349cc55cSDimitry Andric while (Tok != End) { 18781ad6265SDimitry Andric if (!NewText.empty() && !endsWithSpace(NewText)) 188349cc55cSDimitry Andric NewText += " "; 189349cc55cSDimitry Andric 190349cc55cSDimitry Andric NewText += Tok->TokenText; 191349cc55cSDimitry Andric Tok = Tok->Next; 192349cc55cSDimitry Andric } 193349cc55cSDimitry Andric 194349cc55cSDimitry Andric // If we are rotating to the right we move the first token to the back. 195349cc55cSDimitry Andric if (!Left) { 19681ad6265SDimitry Andric if (!NewText.empty() && !startsWithSpace(NewText)) 197349cc55cSDimitry Andric NewText += " "; 198349cc55cSDimitry Andric NewText += First->TokenText; 199349cc55cSDimitry Andric } 200349cc55cSDimitry Andric 201349cc55cSDimitry Andric auto Range = CharSourceRange::getCharRange(First->getStartOfNonWhitespace(), 202349cc55cSDimitry Andric Last->Tok.getEndLoc()); 203349cc55cSDimitry Andric 204349cc55cSDimitry Andric replaceToken(SourceMgr, Fixes, Range, NewText); 205349cc55cSDimitry Andric } 206349cc55cSDimitry Andric 20704eeddc0SDimitry Andric const FormatToken *LeftRightQualifierAlignmentFixer::analyzeRight( 208349cc55cSDimitry Andric const SourceManager &SourceMgr, const AdditionalKeywords &Keywords, 20904eeddc0SDimitry Andric tooling::Replacements &Fixes, const FormatToken *Tok, 210349cc55cSDimitry Andric const std::string &Qualifier, tok::TokenKind QualifierType) { 211349cc55cSDimitry Andric // We only need to think about streams that begin with a qualifier. 212349cc55cSDimitry Andric if (!Tok->is(QualifierType)) 213349cc55cSDimitry Andric return Tok; 214349cc55cSDimitry Andric // Don't concern yourself if nothing follows the qualifier. 215349cc55cSDimitry Andric if (!Tok->Next) 216349cc55cSDimitry Andric return Tok; 217349cc55cSDimitry Andric if (LeftRightQualifierAlignmentFixer::isPossibleMacro(Tok->Next)) 218349cc55cSDimitry Andric return Tok; 219349cc55cSDimitry Andric 22081ad6265SDimitry Andric auto AnalyzeTemplate = 22181ad6265SDimitry Andric [&](const FormatToken *Tok, 22281ad6265SDimitry Andric const FormatToken *StartTemplate) -> const FormatToken * { 22381ad6265SDimitry Andric // Read from the TemplateOpener to TemplateCloser. 22481ad6265SDimitry Andric FormatToken *EndTemplate = StartTemplate->MatchingParen; 22581ad6265SDimitry Andric if (EndTemplate) { 22681ad6265SDimitry Andric // Move to the end of any template class members e.g. 22781ad6265SDimitry Andric // `Foo<int>::iterator`. 22881ad6265SDimitry Andric if (EndTemplate->startsSequence(TT_TemplateCloser, tok::coloncolon, 22981ad6265SDimitry Andric tok::identifier)) { 23081ad6265SDimitry Andric EndTemplate = EndTemplate->Next->Next; 23181ad6265SDimitry Andric } 23281ad6265SDimitry Andric } 23381ad6265SDimitry Andric if (EndTemplate && EndTemplate->Next && 23481ad6265SDimitry Andric !EndTemplate->Next->isOneOf(tok::equal, tok::l_paren)) { 23581ad6265SDimitry Andric insertQualifierAfter(SourceMgr, Fixes, EndTemplate, Qualifier); 23681ad6265SDimitry Andric // Remove the qualifier. 23781ad6265SDimitry Andric removeToken(SourceMgr, Fixes, Tok); 23881ad6265SDimitry Andric return Tok; 23981ad6265SDimitry Andric } 24081ad6265SDimitry Andric return nullptr; 24181ad6265SDimitry Andric }; 24281ad6265SDimitry Andric 243349cc55cSDimitry Andric FormatToken *Qual = Tok->Next; 244349cc55cSDimitry Andric FormatToken *LastQual = Qual; 245349cc55cSDimitry Andric while (Qual && isQualifierOrType(Qual, ConfiguredQualifierTokens)) { 246349cc55cSDimitry Andric LastQual = Qual; 247349cc55cSDimitry Andric Qual = Qual->Next; 248349cc55cSDimitry Andric } 249349cc55cSDimitry Andric if (LastQual && Qual != LastQual) { 250349cc55cSDimitry Andric rotateTokens(SourceMgr, Fixes, Tok, LastQual, /*Left=*/false); 251349cc55cSDimitry Andric Tok = LastQual; 252349cc55cSDimitry Andric } else if (Tok->startsSequence(QualifierType, tok::identifier, 25381ad6265SDimitry Andric TT_TemplateCloser)) { 25481ad6265SDimitry Andric FormatToken *Closer = Tok->Next->Next; 25581ad6265SDimitry Andric rotateTokens(SourceMgr, Fixes, Tok, Tok->Next, /*Left=*/false); 25681ad6265SDimitry Andric Tok = Closer; 257349cc55cSDimitry Andric return Tok; 25881ad6265SDimitry Andric } else if (Tok->startsSequence(QualifierType, tok::identifier, 25981ad6265SDimitry Andric TT_TemplateOpener)) { 26081ad6265SDimitry Andric // `const ArrayRef<int> a;` 26181ad6265SDimitry Andric // `const ArrayRef<int> &a;` 26281ad6265SDimitry Andric const FormatToken *NewTok = AnalyzeTemplate(Tok, Tok->Next->Next); 26381ad6265SDimitry Andric if (NewTok) 26481ad6265SDimitry Andric return NewTok; 26581ad6265SDimitry Andric } else if (Tok->startsSequence(QualifierType, tok::coloncolon, 26681ad6265SDimitry Andric tok::identifier, TT_TemplateOpener)) { 26781ad6265SDimitry Andric // `const ::ArrayRef<int> a;` 26881ad6265SDimitry Andric // `const ::ArrayRef<int> &a;` 26981ad6265SDimitry Andric const FormatToken *NewTok = AnalyzeTemplate(Tok, Tok->Next->Next->Next); 27081ad6265SDimitry Andric if (NewTok) 27181ad6265SDimitry Andric return NewTok; 27281ad6265SDimitry Andric } else if (Tok->startsSequence(QualifierType, tok::identifier) || 27381ad6265SDimitry Andric Tok->startsSequence(QualifierType, tok::coloncolon, 27481ad6265SDimitry Andric tok::identifier)) { 275349cc55cSDimitry Andric FormatToken *Next = Tok->Next; 276349cc55cSDimitry Andric // The case `const Foo` -> `Foo const` 27781ad6265SDimitry Andric // The case `const ::Foo` -> `::Foo const` 278349cc55cSDimitry Andric // The case `const Foo *` -> `Foo const *` 279349cc55cSDimitry Andric // The case `const Foo &` -> `Foo const &` 280349cc55cSDimitry Andric // The case `const Foo &&` -> `Foo const &&` 281349cc55cSDimitry Andric // The case `const std::Foo &&` -> `std::Foo const &&` 282349cc55cSDimitry Andric // The case `const std::Foo<T> &&` -> `std::Foo<T> const &&` 283*1ac55f4cSDimitry Andric // However, `const Bar::*` remains the same. 284*1ac55f4cSDimitry Andric while (Next && Next->isOneOf(tok::identifier, tok::coloncolon) && 285*1ac55f4cSDimitry Andric !Next->startsSequence(tok::coloncolon, tok::star)) { 286349cc55cSDimitry Andric Next = Next->Next; 287*1ac55f4cSDimitry Andric } 288349cc55cSDimitry Andric if (Next && Next->is(TT_TemplateOpener)) { 289349cc55cSDimitry Andric Next = Next->MatchingParen; 290349cc55cSDimitry Andric // Move to the end of any template class members e.g. 291349cc55cSDimitry Andric // `Foo<int>::iterator`. 292349cc55cSDimitry Andric if (Next && Next->startsSequence(TT_TemplateCloser, tok::coloncolon, 29381ad6265SDimitry Andric tok::identifier)) { 294349cc55cSDimitry Andric return Tok; 29581ad6265SDimitry Andric } 296349cc55cSDimitry Andric assert(Next && "Missing template opener"); 297349cc55cSDimitry Andric Next = Next->Next; 298349cc55cSDimitry Andric } 299349cc55cSDimitry Andric if (Next && Next->isOneOf(tok::star, tok::amp, tok::ampamp) && 300349cc55cSDimitry Andric !Tok->Next->isOneOf(Keywords.kw_override, Keywords.kw_final)) { 301349cc55cSDimitry Andric if (Next->Previous && !Next->Previous->is(QualifierType)) { 302349cc55cSDimitry Andric insertQualifierAfter(SourceMgr, Fixes, Next->Previous, Qualifier); 303349cc55cSDimitry Andric removeToken(SourceMgr, Fixes, Tok); 304349cc55cSDimitry Andric } 305349cc55cSDimitry Andric return Next; 306349cc55cSDimitry Andric } 307349cc55cSDimitry Andric } 308349cc55cSDimitry Andric 309349cc55cSDimitry Andric return Tok; 310349cc55cSDimitry Andric } 311349cc55cSDimitry Andric 31204eeddc0SDimitry Andric const FormatToken *LeftRightQualifierAlignmentFixer::analyzeLeft( 313349cc55cSDimitry Andric const SourceManager &SourceMgr, const AdditionalKeywords &Keywords, 31404eeddc0SDimitry Andric tooling::Replacements &Fixes, const FormatToken *Tok, 315349cc55cSDimitry Andric const std::string &Qualifier, tok::TokenKind QualifierType) { 316349cc55cSDimitry Andric // if Tok is an identifier and possibly a macro then don't convert. 317349cc55cSDimitry Andric if (LeftRightQualifierAlignmentFixer::isPossibleMacro(Tok)) 318349cc55cSDimitry Andric return Tok; 319349cc55cSDimitry Andric 32004eeddc0SDimitry Andric const FormatToken *Qual = Tok; 32104eeddc0SDimitry Andric const FormatToken *LastQual = Qual; 322349cc55cSDimitry Andric while (Qual && isQualifierOrType(Qual, ConfiguredQualifierTokens)) { 323349cc55cSDimitry Andric LastQual = Qual; 324349cc55cSDimitry Andric Qual = Qual->Next; 325349cc55cSDimitry Andric if (Qual && Qual->is(QualifierType)) 326349cc55cSDimitry Andric break; 327349cc55cSDimitry Andric } 328349cc55cSDimitry Andric 32981ad6265SDimitry Andric if (!Qual) 330349cc55cSDimitry Andric return Tok; 331349cc55cSDimitry Andric 332349cc55cSDimitry Andric if (LastQual && Qual != LastQual && Qual->is(QualifierType)) { 333349cc55cSDimitry Andric rotateTokens(SourceMgr, Fixes, Tok, Qual, /*Left=*/true); 33481ad6265SDimitry Andric if (!Qual->Next) 33581ad6265SDimitry Andric return Tok; 336349cc55cSDimitry Andric Tok = Qual->Next; 337349cc55cSDimitry Andric } else if (Tok->startsSequence(tok::identifier, QualifierType)) { 338349cc55cSDimitry Andric if (Tok->Next->Next && Tok->Next->Next->isOneOf(tok::identifier, tok::star, 339349cc55cSDimitry Andric tok::amp, tok::ampamp)) { 340349cc55cSDimitry Andric // Don't swap `::iterator const` to `::const iterator`. 341349cc55cSDimitry Andric if (!Tok->Previous || 342349cc55cSDimitry Andric (Tok->Previous && !Tok->Previous->is(tok::coloncolon))) { 343349cc55cSDimitry Andric rotateTokens(SourceMgr, Fixes, Tok, Tok->Next, /*Left=*/true); 344349cc55cSDimitry Andric Tok = Tok->Next; 345349cc55cSDimitry Andric } 34681ad6265SDimitry Andric } else if (Tok->startsSequence(tok::identifier, QualifierType, 34781ad6265SDimitry Andric TT_TemplateCloser)) { 34881ad6265SDimitry Andric FormatToken *Closer = Tok->Next->Next; 34981ad6265SDimitry Andric rotateTokens(SourceMgr, Fixes, Tok, Tok->Next, /*Left=*/true); 35081ad6265SDimitry Andric Tok = Closer; 351349cc55cSDimitry Andric } 352349cc55cSDimitry Andric } 353349cc55cSDimitry Andric if (Tok->is(TT_TemplateOpener) && Tok->Next && 354349cc55cSDimitry Andric (Tok->Next->is(tok::identifier) || Tok->Next->isSimpleTypeSpecifier()) && 355349cc55cSDimitry Andric Tok->Next->Next && Tok->Next->Next->is(QualifierType)) { 356349cc55cSDimitry Andric rotateTokens(SourceMgr, Fixes, Tok->Next, Tok->Next->Next, /*Left=*/true); 357349cc55cSDimitry Andric } 35881ad6265SDimitry Andric if ((Tok->startsSequence(tok::coloncolon, tok::identifier) || 35981ad6265SDimitry Andric Tok->is(tok::identifier)) && 36081ad6265SDimitry Andric Tok->Next) { 361349cc55cSDimitry Andric if (Tok->Previous && 362349cc55cSDimitry Andric Tok->Previous->isOneOf(tok::star, tok::ampamp, tok::amp)) { 363349cc55cSDimitry Andric return Tok; 364349cc55cSDimitry Andric } 36504eeddc0SDimitry Andric const FormatToken *Next = Tok->Next; 366349cc55cSDimitry Andric // The case `std::Foo<T> const` -> `const std::Foo<T> &&` 367349cc55cSDimitry Andric while (Next && Next->isOneOf(tok::identifier, tok::coloncolon)) 368349cc55cSDimitry Andric Next = Next->Next; 369349cc55cSDimitry Andric if (Next && Next->Previous && 370349cc55cSDimitry Andric Next->Previous->startsSequence(tok::identifier, TT_TemplateOpener)) { 371349cc55cSDimitry Andric // Read from to the end of the TemplateOpener to 372349cc55cSDimitry Andric // TemplateCloser const ArrayRef<int> a; const ArrayRef<int> &a; 37304eeddc0SDimitry Andric if (Next->is(tok::comment) && Next->getNextNonComment()) 37404eeddc0SDimitry Andric Next = Next->getNextNonComment(); 375349cc55cSDimitry Andric assert(Next->MatchingParen && "Missing template closer"); 37681ad6265SDimitry Andric Next = Next->MatchingParen; 37781ad6265SDimitry Andric 37881ad6265SDimitry Andric // If the template closer is closing the requires clause, 37981ad6265SDimitry Andric // then stop and go back to the TemplateOpener and do whatever is 38081ad6265SDimitry Andric // inside the <>. 38181ad6265SDimitry Andric if (Next->ClosesRequiresClause) 38281ad6265SDimitry Andric return Next->MatchingParen; 38381ad6265SDimitry Andric Next = Next->Next; 384349cc55cSDimitry Andric 385349cc55cSDimitry Andric // Move to the end of any template class members e.g. 386349cc55cSDimitry Andric // `Foo<int>::iterator`. 387349cc55cSDimitry Andric if (Next && Next->startsSequence(tok::coloncolon, tok::identifier)) 388349cc55cSDimitry Andric Next = Next->Next->Next; 389349cc55cSDimitry Andric if (Next && Next->is(QualifierType)) { 39081ad6265SDimitry Andric // Move the qualifier. 391349cc55cSDimitry Andric insertQualifierBefore(SourceMgr, Fixes, Tok, Qualifier); 392349cc55cSDimitry Andric removeToken(SourceMgr, Fixes, Next); 393349cc55cSDimitry Andric return Next; 394349cc55cSDimitry Andric } 395349cc55cSDimitry Andric } 396349cc55cSDimitry Andric if (Next && Next->Next && 397349cc55cSDimitry Andric Next->Next->isOneOf(tok::amp, tok::ampamp, tok::star)) { 398349cc55cSDimitry Andric if (Next->is(QualifierType)) { 39981ad6265SDimitry Andric // Move the qualifier. 400349cc55cSDimitry Andric insertQualifierBefore(SourceMgr, Fixes, Tok, Qualifier); 401349cc55cSDimitry Andric removeToken(SourceMgr, Fixes, Next); 402349cc55cSDimitry Andric return Next; 403349cc55cSDimitry Andric } 404349cc55cSDimitry Andric } 405349cc55cSDimitry Andric } 406349cc55cSDimitry Andric return Tok; 407349cc55cSDimitry Andric } 408349cc55cSDimitry Andric 409349cc55cSDimitry Andric tok::TokenKind LeftRightQualifierAlignmentFixer::getTokenFromQualifier( 410349cc55cSDimitry Andric const std::string &Qualifier) { 411349cc55cSDimitry Andric // Don't let 'type' be an identifier, but steal typeof token. 412349cc55cSDimitry Andric return llvm::StringSwitch<tok::TokenKind>(Qualifier) 413349cc55cSDimitry Andric .Case("type", tok::kw_typeof) 414349cc55cSDimitry Andric .Case("const", tok::kw_const) 415349cc55cSDimitry Andric .Case("volatile", tok::kw_volatile) 416349cc55cSDimitry Andric .Case("static", tok::kw_static) 417349cc55cSDimitry Andric .Case("inline", tok::kw_inline) 418349cc55cSDimitry Andric .Case("constexpr", tok::kw_constexpr) 419349cc55cSDimitry Andric .Case("restrict", tok::kw_restrict) 420bdd1243dSDimitry Andric .Case("friend", tok::kw_friend) 421349cc55cSDimitry Andric .Default(tok::identifier); 422349cc55cSDimitry Andric } 423349cc55cSDimitry Andric 424349cc55cSDimitry Andric LeftRightQualifierAlignmentFixer::LeftRightQualifierAlignmentFixer( 425349cc55cSDimitry Andric const Environment &Env, const FormatStyle &Style, 426349cc55cSDimitry Andric const std::string &Qualifier, 427349cc55cSDimitry Andric const std::vector<tok::TokenKind> &QualifierTokens, bool RightAlign) 428349cc55cSDimitry Andric : TokenAnalyzer(Env, Style), Qualifier(Qualifier), RightAlign(RightAlign), 429349cc55cSDimitry Andric ConfiguredQualifierTokens(QualifierTokens) {} 430349cc55cSDimitry Andric 431349cc55cSDimitry Andric std::pair<tooling::Replacements, unsigned> 432349cc55cSDimitry Andric LeftRightQualifierAlignmentFixer::analyze( 43381ad6265SDimitry Andric TokenAnnotator & /*Annotator*/, 43481ad6265SDimitry Andric SmallVectorImpl<AnnotatedLine *> &AnnotatedLines, 435349cc55cSDimitry Andric FormatTokenLexer &Tokens) { 436349cc55cSDimitry Andric tooling::Replacements Fixes; 437349cc55cSDimitry Andric const AdditionalKeywords &Keywords = Tokens.getKeywords(); 438349cc55cSDimitry Andric const SourceManager &SourceMgr = Env.getSourceManager(); 439349cc55cSDimitry Andric AffectedRangeMgr.computeAffectedLines(AnnotatedLines); 440349cc55cSDimitry Andric 441349cc55cSDimitry Andric tok::TokenKind QualifierToken = getTokenFromQualifier(Qualifier); 442349cc55cSDimitry Andric assert(QualifierToken != tok::identifier && "Unrecognised Qualifier"); 443349cc55cSDimitry Andric 44404eeddc0SDimitry Andric for (AnnotatedLine *Line : AnnotatedLines) { 44581ad6265SDimitry Andric if (Line->InPPDirective) 44681ad6265SDimitry Andric continue; 44704eeddc0SDimitry Andric FormatToken *First = Line->First; 448d56accc7SDimitry Andric assert(First); 449d56accc7SDimitry Andric if (First->Finalized) 450d56accc7SDimitry Andric continue; 451d56accc7SDimitry Andric 45204eeddc0SDimitry Andric const auto *Last = Line->Last; 453349cc55cSDimitry Andric 45404eeddc0SDimitry Andric for (const auto *Tok = First; Tok && Tok != Last && Tok->Next; 45504eeddc0SDimitry Andric Tok = Tok->Next) { 456349cc55cSDimitry Andric if (Tok->is(tok::comment)) 457349cc55cSDimitry Andric continue; 45881ad6265SDimitry Andric if (RightAlign) { 459349cc55cSDimitry Andric Tok = analyzeRight(SourceMgr, Keywords, Fixes, Tok, Qualifier, 460349cc55cSDimitry Andric QualifierToken); 46181ad6265SDimitry Andric } else { 462349cc55cSDimitry Andric Tok = analyzeLeft(SourceMgr, Keywords, Fixes, Tok, Qualifier, 463349cc55cSDimitry Andric QualifierToken); 464349cc55cSDimitry Andric } 465349cc55cSDimitry Andric } 46681ad6265SDimitry Andric } 467349cc55cSDimitry Andric return {Fixes, 0}; 468349cc55cSDimitry Andric } 469349cc55cSDimitry Andric 470349cc55cSDimitry Andric void QualifierAlignmentFixer::PrepareLeftRightOrdering( 471349cc55cSDimitry Andric const std::vector<std::string> &Order, std::vector<std::string> &LeftOrder, 472349cc55cSDimitry Andric std::vector<std::string> &RightOrder, 473349cc55cSDimitry Andric std::vector<tok::TokenKind> &Qualifiers) { 474349cc55cSDimitry Andric 475349cc55cSDimitry Andric // Depending on the position of type in the order you need 476349cc55cSDimitry Andric // To iterate forward or backward through the order list as qualifier 477349cc55cSDimitry Andric // can push through each other. 478349cc55cSDimitry Andric // The Order list must define the position of "type" to signify 479349cc55cSDimitry Andric assert(llvm::is_contained(Order, "type") && 480349cc55cSDimitry Andric "QualifierOrder must contain type"); 481349cc55cSDimitry Andric // Split the Order list by type and reverse the left side. 482349cc55cSDimitry Andric 483349cc55cSDimitry Andric bool left = true; 484349cc55cSDimitry Andric for (const auto &s : Order) { 485349cc55cSDimitry Andric if (s == "type") { 486349cc55cSDimitry Andric left = false; 487349cc55cSDimitry Andric continue; 488349cc55cSDimitry Andric } 489349cc55cSDimitry Andric 490349cc55cSDimitry Andric tok::TokenKind QualifierToken = 491349cc55cSDimitry Andric LeftRightQualifierAlignmentFixer::getTokenFromQualifier(s); 49281ad6265SDimitry Andric if (QualifierToken != tok::kw_typeof && QualifierToken != tok::identifier) 493349cc55cSDimitry Andric Qualifiers.push_back(QualifierToken); 494349cc55cSDimitry Andric 49581ad6265SDimitry Andric if (left) { 496349cc55cSDimitry Andric // Reverse the order for left aligned items. 497349cc55cSDimitry Andric LeftOrder.insert(LeftOrder.begin(), s); 49881ad6265SDimitry Andric } else { 499349cc55cSDimitry Andric RightOrder.push_back(s); 500349cc55cSDimitry Andric } 501349cc55cSDimitry Andric } 50281ad6265SDimitry Andric } 503349cc55cSDimitry Andric 504349cc55cSDimitry Andric bool LeftRightQualifierAlignmentFixer::isQualifierOrType( 505349cc55cSDimitry Andric const FormatToken *Tok, const std::vector<tok::TokenKind> &specifiedTypes) { 506349cc55cSDimitry Andric return Tok && (Tok->isSimpleTypeSpecifier() || Tok->is(tok::kw_auto) || 507349cc55cSDimitry Andric llvm::is_contained(specifiedTypes, Tok->Tok.getKind())); 508349cc55cSDimitry Andric } 509349cc55cSDimitry Andric 510349cc55cSDimitry Andric // If a token is an identifier and it's upper case, it could 511349cc55cSDimitry Andric // be a macro and hence we need to be able to ignore it. 512349cc55cSDimitry Andric bool LeftRightQualifierAlignmentFixer::isPossibleMacro(const FormatToken *Tok) { 513349cc55cSDimitry Andric if (!Tok) 514349cc55cSDimitry Andric return false; 515349cc55cSDimitry Andric if (!Tok->is(tok::identifier)) 516349cc55cSDimitry Andric return false; 51781ad6265SDimitry Andric if (Tok->TokenText.upper() == Tok->TokenText.str()) { 51881ad6265SDimitry Andric // T,K,U,V likely could be template arguments 51981ad6265SDimitry Andric return (Tok->TokenText.size() != 1); 52081ad6265SDimitry Andric } 521349cc55cSDimitry Andric return false; 522349cc55cSDimitry Andric } 523349cc55cSDimitry Andric 524349cc55cSDimitry Andric } // namespace format 525349cc55cSDimitry Andric } // namespace clang 526