[PATCH] D25826: [ELF] Show error location for 'undefined symbol' errors

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 20 08:10:21 PDT 2016


grimar added inline comments.


================
Comment at: ELF/InputFiles.cpp:411
+    auto *D = dyn_cast<DefinedRegular<ELFT>>(B);
+    if (!D)
+      continue;
----------------
May be 

```
if (!D || D->Section != S)
```


================
Comment at: ELF/Relocations.cpp:543
+  return (SrcFile + " (" + S.Name + "+0x" + Twine::utohexstr(Offset) + ")")
+      .str();
+}
----------------
Looks you do not need .str() here.


https://reviews.llvm.org/D25826





More information about the llvm-commits mailing list