[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