[PATCH] D74773: [LLD][Debuginfo][NFC] create DWARFContext only once for the same object file.

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 24 05:46:40 PST 2020


jhenderson added inline comments.


================
Comment at: lld/ELF/SyntheticSections.cpp:2671
     if (Error e = cu->tryExtractDIEsIfNeeded(false)) {
-      error(toString(sec) + ": " + toString(std::move(e)));
+      warn(toString(sec) + ": " + toString(std::move(e)));
       return {};
----------------
Seems like a not-really-related change? It's certainly not NFC.


================
Comment at: lld/ELF/SyntheticSections.cpp:2676
     if (!ranges) {
-      error(toString(sec) + ": " + toString(ranges.takeError()));
+      warn(toString(sec) + ": " + toString(ranges.takeError()));
       return {};
----------------
Ditto.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74773





More information about the llvm-commits mailing list