[PATCH] D59311: [ELF] Dump symbols ordered by profiled guided section layout to file.

[Michael Spencer via Phabricator via llvm-commits]

Bigcheese added inline comments.


================
Comment at: lld/ELF/Driver.cpp:1475
     error("cannot open map file " + Config->MapFile + ": " + E.message());
+  // Fail early too if the symbol order file is not writable
+  if (auto E = tryCreateFile(Config->PrintSymbolOrder))
----------------
ruiu wrote:
> I don't think we need this guard, as this is a minor feature. Can you remove this code?
Why not be consistent here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D59311