This is an archive of the discontinued LLVM Phabricator instance.

[NFC] Introduce a `LateInitialize()` method to `SymbolizerTool` that is called during the LateInitialize stage of the sanitizer runtimes.
ClosedPublic

Authored by delcypher on Apr 14 2020, 10:43 PM.

Details

Summary

This is implemented by adding a Symbolizer::LateInitializeTools()
method that iterates over the registered tools and calls the
LateInitialize() method on them.

Symbolizer::LateInitializeTools() is now called from the various
Symbolizer::LateInitialize() implementations.

The default implementation of SymbolizerTool::LateInitialize()
does nothing so this change should be NFC.

This change allows SymbolizerTool implementations to perform
any initialization that they need to perform at the
LateInitialize stage of a sanitizer runtime init.

rdar://problem/58789439

Diff Detail

Event Timeline

delcypher created this revision.Apr 14 2020, 10:43 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 14 2020, 10:43 PM
Herald added a subscriber: Restricted Project. · View Herald Transcript
delcypher updated this revision to Diff 257944.Apr 15 2020, 7:42 PM
  • Update radar number
delcypher edited the summary of this revision. (Show Details)Apr 15 2020, 7:44 PM
delcypher updated this revision to Diff 258150.Apr 16 2020, 1:06 PM
  • Symbolizer::LateInitializeTools() should be private.
delcypher updated this revision to Diff 258151.Apr 16 2020, 1:10 PM

Tweak comments.

vitalybuka accepted this revision.Apr 16 2020, 3:28 PM
This revision is now accepted and ready to land.Apr 16 2020, 3:28 PM

@vitalybuka Thanks for the review.

This revision was automatically updated to reflect the committed changes.