// simply avoid always having the same function names, and we need to
// remain deterministic.
unsigned int randSeed = 0;
- for (std::string::const_iterator I = M.getModuleIdentifier().begin(),
- E = M.getModuleIdentifier().end(); I != E; ++I)
- randSeed += *I;
+ for (auto C : M.getModuleIdentifier())
+ randSeed += C;
Renamer renamer(randSeed);
// Rename all aliases
- for (Module::alias_iterator AI = M.alias_begin(), AE = M.alias_end();
- AI != AE; ++AI) {
+ for (auto AI = M.alias_begin(), AE = M.alias_end(); AI != AE; ++AI) {
StringRef Name = AI->getName();
if (Name.startswith("llvm.") || (!Name.empty() && Name[0] == 1))
continue;
}
// Rename all global variables
- for (Module::global_iterator GI = M.global_begin(), GE = M.global_end();
- GI != GE; ++GI) {
+ for (auto GI = M.global_begin(), GE = M.global_end(); GI != GE; ++GI) {
StringRef Name = GI->getName();
if (Name.startswith("llvm.") || (!Name.empty() && Name[0] == 1))
continue;
// Rename all struct types
TypeFinder StructTypes;
StructTypes.run(M, true);
- for (unsigned i = 0, e = StructTypes.size(); i != e; ++i) {
- StructType *STy = StructTypes[i];
+ for (StructType *STy : StructTypes) {
if (STy->isLiteral() || STy->getName().empty()) continue;
SmallString<128> NameStorage;
}
// Rename all functions
- for (Module::iterator FI = M.begin(), FE = M.end();
- FI != FE; ++FI) {
- StringRef Name = FI->getName();
+ for (auto &F : M) {
+ StringRef Name = F.getName();
if (Name.startswith("llvm.") || (!Name.empty() && Name[0] == 1))
continue;
- FI->setName(renamer.newName());
- runOnFunction(*FI);
+ F.setName(renamer.newName());
+ runOnFunction(F);
}
return true;
}
bool runOnFunction(Function &F) {
- for (Function::arg_iterator AI = F.arg_begin(), AE = F.arg_end();
- AI != AE; ++AI)
+ for (auto AI = F.arg_begin(), AE = F.arg_end(); AI != AE; ++AI)
if (!AI->getType()->isVoidTy())
AI->setName("arg");
- for (Function::iterator BB = F.begin(), E = F.end(); BB != E; ++BB) {
- BB->setName("bb");
+ for (auto &BB : F) {
+ BB.setName("bb");
- for (BasicBlock::iterator I = BB->begin(), E = BB->end(); I != E; ++I)
- if (!I->getType()->isVoidTy())
- I->setName("tmp");
+ for (auto &I : BB)
+ if (!I.getType()->isVoidTy())
+ I.setName("tmp");
}
return true;
}