[PATCH] D78530: Add missing call to `Symbolizer::LateInitialize()` in UBSan's standalone init.

Dan Liew via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 21 11:56:18 PDT 2020


delcypher added a comment.

@vsk @vitalybuka Any thoughts on this?

If I was really being consistent with ASan/TSan init I would **also add a call to `Symbolizer::GetOrInit()` before `Symbolizer::LateInitialize()`** but it's technically not necessary because all `Symbolizer::LateInitialize()` implementations call it anyway. To keep the patch simple I didn't add it but should I for consistency?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78530





More information about the llvm-commits mailing list