Address review comments

Created using spr 1.3.6-beta.1

Author: pcc

llvm/lib/LTO/LTO.cpp

@@ -871,10 +871,10 @@ LTO::addRegularLTO(InputFile &Input, ArrayRef<SymbolResolution> InputRes,
   } else if (NamedMDNode *AliasesMD = M.getNamedMetadata("aliases")) {
     // Delete aliases entries for non-prevailing symbols on the ThinLTO side of
     // this input file.
-    std::set<StringRef> Prevailing;
-    for (auto P : zip(Input.symbols(), Res))
-      if (std::get<1>(P).Prevailing && !std::get<0>(P).getIRName().empty())
-        Prevailing.insert(std::get<0>(P).getIRName());
+    DenseSet<StringRef> Prevailing;
+    for (auto [I, R] : zip(Input.symbols(), Res))
+      if (R.Prevailing && !I.getIRName().empty())
+        Prevailing.insert(I.getIRName());
     std::vector<MDNode *> AliasGroups;
     for (MDNode *AliasGroup : AliasesMD->operands()) {
       std::vector<Metadata *> Aliases;

llvm/lib/Transforms/IPO/LowerTypeTests.cpp

@@ -2245,7 +2245,7 @@ bool LowerTypeTestsModule::lower() {
   if (ExportSummary) {
     if (NamedMDNode *AliasesMD = M.getNamedMetadata("aliases")) {
       for (auto *AliasMD : AliasesMD->operands()) {
-        std::vector<Function *> Aliases;
+        SmallVector<Function *> Aliases;
         for (Metadata *MD : AliasMD->operands()) {
           auto *MDS = dyn_cast<MDString>(MD);
           if (!MDS)

llvm/lib/Transforms/IPO/ThinLTOBitcodeWriter.cpp

@@ -444,7 +444,7 @@ void splitAndWriteThinLTOBitcode(
   if (!FunctionAliases.empty()) {
     NamedMDNode *NMD = MergedM->getOrInsertNamedMetadata("aliases");
     for (auto &Alias : FunctionAliases) {
-      std::vector<Metadata *> Elts;
+      SmallVector<Metadata *> Elts;
       Elts.push_back(MDString::get(Ctx, Alias.first->getName()));
       for (auto *A : Alias.second)
         Elts.push_back(MDString::get(Ctx, A->getName()));