[PATCH] D128184: [lld-macho] Show source information for undefined references

[Daniel Bertalan via Phabricator via llvm-commits]

BertalanD added inline comments.


================
Comment at: lld/MachO/InputFiles.cpp:1379
+std::string ObjFile::sourceFile() const {
+  if (!compileUnit)
+    return {};
----------------
thakis wrote:
> Do we need this check? All current callers of sourceFile() check compileUnit before calling this.
This sounds like one of those narrow vs wide contract API questions. Should I remove the check from here, replace it with an assert, or remove the check from `getSourceLocation`? I don't really have a preference.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D128184/new/

https://reviews.llvm.org/D128184