summaryrefslogtreecommitdiffstats
path: root/clang-tools-extra/modularize
diff options
context:
space:
mode:
Diffstat (limited to 'clang-tools-extra/modularize')
-rw-r--r--clang-tools-extra/modularize/CoverageChecker.cpp5
-rw-r--r--clang-tools-extra/modularize/Modularize.cpp4
-rw-r--r--clang-tools-extra/modularize/ModularizeUtilities.cpp7
-rw-r--r--clang-tools-extra/modularize/ModuleAssistant.cpp15
-rw-r--r--clang-tools-extra/modularize/PreprocessorTracker.cpp70
5 files changed, 38 insertions, 63 deletions
diff --git a/clang-tools-extra/modularize/CoverageChecker.cpp b/clang-tools-extra/modularize/CoverageChecker.cpp
index b3aa3c9b79e..630ab558c5f 100644
--- a/clang-tools-extra/modularize/CoverageChecker.cpp
+++ b/clang-tools-extra/modularize/CoverageChecker.cpp
@@ -225,9 +225,8 @@ bool CoverageChecker::collectModuleHeaders(const Module &Mod) {
ModuleMapHeadersSet.insert(ModularizeUtilities::getCanonicalPath(
Header.Entry->getName()));
- for (Module::submodule_const_iterator MI = Mod.submodule_begin(),
- MIEnd = Mod.submodule_end();
- MI != MIEnd; ++MI)
+ for (auto MI = Mod.submodule_begin(), MIEnd = Mod.submodule_end();
+ MI != MIEnd; ++MI)
collectModuleHeaders(**MI);
return true;
diff --git a/clang-tools-extra/modularize/Modularize.cpp b/clang-tools-extra/modularize/Modularize.cpp
index 284e3299894..83f23402ba2 100644
--- a/clang-tools-extra/modularize/Modularize.cpp
+++ b/clang-tools-extra/modularize/Modularize.cpp
@@ -341,9 +341,7 @@ static std::string findInputFile(const CommandLineArguments &CLArgs) {
const unsigned IncludedFlagsBitmask = options::CC1Option;
unsigned MissingArgIndex, MissingArgCount;
SmallVector<const char *, 256> Argv;
- for (CommandLineArguments::const_iterator I = CLArgs.begin(),
- E = CLArgs.end();
- I != E; ++I)
+ for (auto I = CLArgs.begin(), E = CLArgs.end(); I != E; ++I)
Argv.push_back(I->c_str());
InputArgList Args = Opts->ParseArgs(Argv, MissingArgIndex, MissingArgCount,
IncludedFlagsBitmask);
diff --git a/clang-tools-extra/modularize/ModularizeUtilities.cpp b/clang-tools-extra/modularize/ModularizeUtilities.cpp
index 9a268950529..f20f4f6b782 100644
--- a/clang-tools-extra/modularize/ModularizeUtilities.cpp
+++ b/clang-tools-extra/modularize/ModularizeUtilities.cpp
@@ -77,7 +77,7 @@ ModularizeUtilities *ModularizeUtilities::createModularizeUtilities(
std::error_code ModularizeUtilities::loadAllHeaderListsAndDependencies() {
typedef std::vector<std::string>::iterator Iter;
// For each input file.
- for (Iter I = InputFilePaths.begin(), E = InputFilePaths.end(); I != E; ++I) {
+ for (auto I = InputFilePaths.begin(), E = InputFilePaths.end(); I != E; ++I) {
llvm::StringRef InputPath = *I;
// If it's a module map.
if (InputPath.endswith(".modulemap")) {
@@ -345,9 +345,8 @@ bool ModularizeUtilities::collectModuleHeaders(const clang::Module &Mod) {
DependentsVector UmbrellaDependents;
// Recursively do submodules.
- for (clang::Module::submodule_const_iterator MI = Mod.submodule_begin(),
- MIEnd = Mod.submodule_end();
- MI != MIEnd; ++MI)
+ for (auto MI = Mod.submodule_begin(), MIEnd = Mod.submodule_end();
+ MI != MIEnd; ++MI)
collectModuleHeaders(**MI);
if (const FileEntry *UmbrellaHeader = Mod.getUmbrellaHeader().Entry) {
diff --git a/clang-tools-extra/modularize/ModuleAssistant.cpp b/clang-tools-extra/modularize/ModuleAssistant.cpp
index e1c32dfaa82..26c592540a6 100644
--- a/clang-tools-extra/modularize/ModuleAssistant.cpp
+++ b/clang-tools-extra/modularize/ModuleAssistant.cpp
@@ -87,17 +87,14 @@ bool Module::output(llvm::raw_fd_ostream &OS, int Indent) {
}
// Output submodules.
- for (std::vector<Module *>::iterator I = SubModules.begin(),
- E = SubModules.end();
- I != E; ++I) {
+ for (auto I = SubModules.begin(), E = SubModules.end(); I != E; ++I) {
if (!(*I)->output(OS, Indent))
return false;
}
// Output header files.
- for (std::vector<std::string>::iterator I = HeaderFileNames.begin(),
- E = HeaderFileNames.end();
- I != E; ++I) {
+ for (auto I = HeaderFileNames.begin(), E = HeaderFileNames.end(); I != E;
+ ++I) {
OS.indent(Indent);
if (IsProblem || strstr((*I).c_str(), ".inl"))
OS << "exclude header \"" << *I << "\"\n";
@@ -123,9 +120,7 @@ bool Module::output(llvm::raw_fd_ostream &OS, int Indent) {
// Lookup a sub-module.
Module *Module::findSubModule(llvm::StringRef SubName) {
- for (std::vector<Module *>::iterator I = SubModules.begin(),
- E = SubModules.end();
- I != E; ++I) {
+ for (auto I = SubModules.begin(), E = SubModules.end(); I != E; ++I) {
if ((*I)->Name == SubName)
return *I;
}
@@ -227,7 +222,7 @@ static Module *loadModuleDescriptions(
DependencyMap &Dependencies, llvm::StringRef HeaderPrefix) {
// Create root module.
- Module *RootModule = new Module(RootModuleName, false);
+ auto *RootModule = new Module(RootModuleName, false);
llvm::SmallString<256> CurrentDirectory;
llvm::sys::fs::current_path(CurrentDirectory);
diff --git a/clang-tools-extra/modularize/PreprocessorTracker.cpp b/clang-tools-extra/modularize/PreprocessorTracker.cpp
index f594cd5a7ee..319356897b1 100644
--- a/clang-tools-extra/modularize/PreprocessorTracker.cpp
+++ b/clang-tools-extra/modularize/PreprocessorTracker.cpp
@@ -558,9 +558,7 @@ public:
// Check for the presence of a header inclusion path handle entry.
// Return false if not found.
bool haveInclusionPathHandle(InclusionPathHandle H) {
- for (std::vector<InclusionPathHandle>::iterator
- I = InclusionPathHandles.begin(),
- E = InclusionPathHandles.end();
+ for (auto I = InclusionPathHandles.begin(), E = InclusionPathHandles.end();
I != E; ++I) {
if (*I == H)
return true;
@@ -608,9 +606,8 @@ public:
MacroExpansionInstance *
findMacroExpansionInstance(StringHandle MacroExpanded,
PPItemKey &DefinitionLocation) {
- for (std::vector<MacroExpansionInstance>::iterator
- I = MacroExpansionInstances.begin(),
- E = MacroExpansionInstances.end();
+ for (auto I = MacroExpansionInstances.begin(),
+ E = MacroExpansionInstances.end();
I != E; ++I) {
if ((I->MacroExpanded == MacroExpanded) &&
(I->DefinitionLocation == DefinitionLocation)) {
@@ -659,9 +656,7 @@ public:
// Check for the presence of a header inclusion path handle entry.
// Return false if not found.
bool haveInclusionPathHandle(InclusionPathHandle H) {
- for (std::vector<InclusionPathHandle>::iterator
- I = InclusionPathHandles.begin(),
- E = InclusionPathHandles.end();
+ for (auto I = InclusionPathHandles.begin(), E = InclusionPathHandles.end();
I != E; ++I) {
if (*I == H)
return true;
@@ -701,9 +696,8 @@ public:
// Find a matching condition expansion instance.
ConditionalExpansionInstance *
findConditionalExpansionInstance(clang::PPCallbacks::ConditionValueKind ConditionValue) {
- for (std::vector<ConditionalExpansionInstance>::iterator
- I = ConditionalExpansionInstances.begin(),
- E = ConditionalExpansionInstances.end();
+ for (auto I = ConditionalExpansionInstances.begin(),
+ E = ConditionalExpansionInstances.end();
I != E; ++I) {
if (I->ConditionValue == ConditionValue) {
return &*I; // Found.
@@ -954,9 +948,8 @@ public:
HeaderHandle findHeaderHandle(llvm::StringRef HeaderPath) const {
std::string CanonicalPath = getCanonicalPath(HeaderPath);
HeaderHandle H = 0;
- for (std::vector<StringHandle>::const_iterator I = HeaderPaths.begin(),
- E = HeaderPaths.end();
- I != E; ++I, ++H) {
+ for (auto I = HeaderPaths.begin(), E = HeaderPaths.end(); I != E;
+ ++I, ++H) {
if (**I == CanonicalPath)
return H;
}
@@ -1004,9 +997,7 @@ public:
// Check for presence of header handle in the header stack.
bool isHeaderHandleInStack(HeaderHandle H) const {
- for (std::vector<HeaderHandle>::const_iterator I = HeaderStack.begin(),
- E = HeaderStack.end();
- I != E; ++I) {
+ for (auto I = HeaderStack.begin(), E = HeaderStack.end(); I != E; ++I) {
if (*I == H)
return true;
}
@@ -1018,10 +1009,8 @@ public:
InclusionPathHandle
findInclusionPathHandle(const std::vector<HeaderHandle> &Path) const {
InclusionPathHandle H = 0;
- for (std::vector<HeaderInclusionPath>::const_iterator
- I = InclusionPaths.begin(),
- E = InclusionPaths.end();
- I != E; ++I, ++H) {
+ for (auto I = InclusionPaths.begin(), E = InclusionPaths.end(); I != E;
+ ++I, ++H) {
if (I->Path == Path)
return H;
}
@@ -1065,7 +1054,7 @@ public:
StringHandle MacroName = addString(II->getName());
PPItemKey InstanceKey(PP, MacroName, H, InstanceLoc);
PPItemKey DefinitionKey(PP, MacroName, H, DefinitionLoc);
- MacroExpansionMapIter I = MacroExpansions.find(InstanceKey);
+ auto I = MacroExpansions.find(InstanceKey);
// If existing instance of expansion not found, add one.
if (I == MacroExpansions.end()) {
std::string InstanceSourceLine =
@@ -1113,7 +1102,7 @@ public:
return;
StringHandle ConditionUnexpandedHandle(addString(ConditionUnexpanded));
PPItemKey InstanceKey(PP, ConditionUnexpandedHandle, H, InstanceLoc);
- ConditionalExpansionMapIter I = ConditionalExpansions.find(InstanceKey);
+ auto I = ConditionalExpansions.find(InstanceKey);
// If existing instance of condition not found, add one.
if (I == ConditionalExpansions.end()) {
std::string InstanceSourceLine =
@@ -1144,9 +1133,8 @@ public:
bool reportInconsistentMacros(llvm::raw_ostream &OS) override {
bool ReturnValue = false;
// Walk all the macro expansion trackers in the map.
- for (MacroExpansionMapIter I = MacroExpansions.begin(),
- E = MacroExpansions.end();
- I != E; ++I) {
+ for (auto I = MacroExpansions.begin(), E = MacroExpansions.end(); I != E;
+ ++I) {
const PPItemKey &ItemKey = I->first;
MacroExpansionTracker &MacroExpTracker = I->second;
// If no mismatch (only one instance value) continue.
@@ -1162,21 +1150,19 @@ public:
<< "' has different values in this header, depending on how it was "
"included.\n";
// Walk all the instances.
- for (std::vector<MacroExpansionInstance>::iterator
- IMT = MacroExpTracker.MacroExpansionInstances.begin(),
- EMT = MacroExpTracker.MacroExpansionInstances.end();
+ for (auto IMT = MacroExpTracker.MacroExpansionInstances.begin(),
+ EMT = MacroExpTracker.MacroExpansionInstances.end();
IMT != EMT; ++IMT) {
MacroExpansionInstance &MacroInfo = *IMT;
OS << " '" << *MacroExpTracker.MacroUnexpanded << "' expanded to: '"
<< *MacroInfo.MacroExpanded
<< "' with respect to these inclusion paths:\n";
// Walk all the inclusion path hierarchies.
- for (std::vector<InclusionPathHandle>::iterator
- IIP = MacroInfo.InclusionPathHandles.begin(),
- EIP = MacroInfo.InclusionPathHandles.end();
+ for (auto IIP = MacroInfo.InclusionPathHandles.begin(),
+ EIP = MacroInfo.InclusionPathHandles.end();
IIP != EIP; ++IIP) {
const std::vector<HeaderHandle> &ip = getInclusionPath(*IIP);
- int Count = (int)ip.size();
+ auto Count = (int)ip.size();
for (int Index = 0; Index < Count; ++Index) {
HeaderHandle H = ip[Index];
OS << std::string((Index * 2) + 4, ' ') << *getHeaderFilePath(H)
@@ -1205,8 +1191,8 @@ public:
bool reportInconsistentConditionals(llvm::raw_ostream &OS) override {
bool ReturnValue = false;
// Walk all the conditional trackers in the map.
- for (ConditionalExpansionMapIter I = ConditionalExpansions.begin(),
- E = ConditionalExpansions.end();
+ for (auto I = ConditionalExpansions.begin(),
+ E = ConditionalExpansions.end();
I != E; ++I) {
const PPItemKey &ItemKey = I->first;
ConditionalTracker &CondTracker = I->second;
@@ -1225,21 +1211,19 @@ public:
<< "' has different values in this header, depending on how it was "
"included.\n";
// Walk all the instances.
- for (std::vector<ConditionalExpansionInstance>::iterator
- IMT = CondTracker.ConditionalExpansionInstances.begin(),
- EMT = CondTracker.ConditionalExpansionInstances.end();
+ for (auto IMT = CondTracker.ConditionalExpansionInstances.begin(),
+ EMT = CondTracker.ConditionalExpansionInstances.end();
IMT != EMT; ++IMT) {
ConditionalExpansionInstance &MacroInfo = *IMT;
OS << " '" << *CondTracker.ConditionUnexpanded << "' expanded to: '"
<< ConditionValueKindStrings[MacroInfo.ConditionValue]
<< "' with respect to these inclusion paths:\n";
// Walk all the inclusion path hierarchies.
- for (std::vector<InclusionPathHandle>::iterator
- IIP = MacroInfo.InclusionPathHandles.begin(),
- EIP = MacroInfo.InclusionPathHandles.end();
+ for (auto IIP = MacroInfo.InclusionPathHandles.begin(),
+ EIP = MacroInfo.InclusionPathHandles.end();
IIP != EIP; ++IIP) {
const std::vector<HeaderHandle> &ip = getInclusionPath(*IIP);
- int Count = (int)ip.size();
+ auto Count = (int)ip.size();
for (int Index = 0; Index < Count; ++Index) {
HeaderHandle H = ip[Index];
OS << std::string((Index * 2) + 4, ' ') << *getHeaderFilePath(H)
OpenPOWER on IntegriCloud