Replace a std::pair with a struct.
authorRafael Espindola <rafael.espindola@gmail.com>
Wed, 21 Mar 2018 22:32:17 +0000 (22:32 +0000)
committerRafael Espindola <rafael.espindola@gmail.com>
Wed, 21 Mar 2018 22:32:17 +0000 (22:32 +0000)
This is more readable and should reduce the noise in a followup patch.

llvm-svn: 328164

lld/ELF/InputFiles.cpp
lld/ELF/InputFiles.h

index d69a40c..514bc77 100644 (file)
@@ -183,11 +183,11 @@ ObjFile<ELFT>::getVariableLoc(StringRef Name) {
   // Take file name string from line table.
   std::string FileName;
   if (!LT->getFileNameByIndex(
-          It->second.first /* File */, nullptr,
+          It->second.File, nullptr,
           DILineInfoSpecifier::FileLineInfoKind::AbsoluteFilePath, FileName))
     return None;
 
-  return std::make_pair(FileName, It->second.second /*Line*/);
+  return std::make_pair(FileName, It->second.Line);
 }
 
 // Returns source line information for a given offset
index 42ae4e4..8b2c853 100644 (file)
@@ -216,7 +216,11 @@ private:
   // single object file, so we cache debugging information in order to
   // parse it only once for each object file we link.
   std::unique_ptr<llvm::DWARFDebugLine> DwarfLine;
-  llvm::DenseMap<StringRef, std::pair<unsigned, unsigned>> VariableLoc;
+  struct VarLoc {
+    unsigned File;
+    unsigned Line;
+  };
+  llvm::DenseMap<StringRef, VarLoc> VariableLoc;
   llvm::once_flag InitDwarfLine;
 };