replica
/
hanchenye-llvm-project
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

SourceRanges are small and trivially copyable, don't them by reference. NFC 

llvm-svn: 249258
This commit is contained in:
Craig Topper 2015-10-04 04:53:37 +00:00
parent 161935520d
commit 8f70a9f892
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
clang-tools-extra/clang-modernize/LoopConvert/LoopActions.cpp
View File

@ -811,7 +811,7 @@ void LoopFixer::doConversion(ASTContext *Context,
AliasVarIsRef = AliasVar->getType()->isReferenceType();
// We keep along the entire DeclStmt to keep the correct range here.
const SourceRange &ReplaceRange = AliasDecl->getSourceRange();
SourceRange ReplaceRange = AliasDecl->getSourceRange();
std::string ReplacementText;
if (AliasUseRequired)

4
clang-tools-extra/clang-tidy/utils/IncludeSorter.cpp
View File

@ -243,11 +243,11 @@ std::vector<FixItHint> IncludeSorter::GetEdits() {
std::string CurrentText;
std::vector<FixItHint> Fixes;
for (const auto &LineEdit : Edits) {
const SourceRange &EditRange = LineEdit.second.first;
// If the current edit is on the next line after the previous edit, add it
// to the current block edit.
if (LineEdit.first == CurrentEndLine + 1 &&
CurrentRange.getBegin() != CurrentRange.getEnd()) {
SourceRange EditRange = LineEdit.second.first;
if (EditRange.getBegin() != EditRange.getEnd()) {
++CurrentEndLine;
CurrentRange.setEnd(EditRange.getEnd());
@ -260,7 +260,7 @@ std::vector<FixItHint> IncludeSorter::GetEdits() {
}
CurrentEndLine = LineEdit.first;
CurrentRange = EditRange;
CurrentRange = LineEdit.second.first;
CurrentText = LineEdit.second.second;
}
}