[PATCH] D91765: [sanitizer_common] Add facility to get the full report path

[Teresa Johnson via Phabricator via llvm-commits]

tejohnson added a comment.

In D91765#2404679 <https://reviews.llvm.org/D91765#2404679>, @vitalybuka wrote:

> It mentions the test, but it's not here

Woops, forgot to git add it. It's there now.



================
Comment at: compiler-rt/lib/sanitizer_common/sanitizer_file.cpp:100
+  SpinMutexLock l(mu);
+  ReopenIfNecessary();
+  return full_path;
----------------
vitalybuka wrote:
> why reopen is needed?
Because the actual full_path (with pid suffix) is populated only on open.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91765