[PATCH] D118665: [Debuginfod] Flag-determine debuginfod lookups in llvm-symbolizer.

[Petr Hosek via Phabricator via llvm-commits]

phosek accepted this revision.
phosek added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: llvm/tools/llvm-symbolizer/llvm-symbolizer.cpp:273-276
+  // A debuginfod lookup could also succeed if something were present in the
+  // cache directory, but it would be surprising to enable debuginfod on this
+  // basis alone. To use existing caches in an "offline" fashion, the debuginfod
+  // flag must be set.
----------------
This is just a nit, but I find that having the comment here not followed by any code a bit confusing, especially since it's referring to logic inside `main`. I'd slightly prefer just inlining this function together with these comments into `main`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118665