[PATCH] D97579: [compiler-rt] Silence warnings about large stack frames in ReadBinaryName

[Reid Kleckner via Phabricator via llvm-commits]

rnk added a comment.

I dug back to 2014 to find out why we have this flag: rG167c15a98feb898c13eca325678b73d0c59dffa2 <https://reviews.llvm.org/rG167c15a98feb898c13eca325678b73d0c59dffa2> Unfortunately, it seems there is no rationale. I'm guessing it's carried over from TSan, which has this flag. TSan uses sanitizer_common, so if tsan cares about frame size, then sanitizer_common does. The TSan warning flag comes from rG06379b35373e0ca9561c130284104fbdf0295f79 <https://reviews.llvm.org/rG06379b35373e0ca9561c130284104fbdf0295f79>, but again, no rationale or link to a review thread. =/

Instead of disabling the warning and adding ifdefs, maybe it's better to just fix the warning with `InternalMmapVector<wchar_t>`. The vast majority of uses of kMaxPathLength are for local InternalScopedString variables.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97579