This is an archive of the discontinued LLVM Phabricator instance.

[sanitizer] Remove id and replace link with u32
ClosedPublic

Authored by vitalybuka on Oct 12 2021, 12:06 AM.

Details

Summary

This lets us reduce size of Node, similar to D111183 proposal.

Depends on D111610.

Diff Detail

Event Timeline

vitalybuka requested review of this revision.Oct 12 2021, 12:06 AM
vitalybuka created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptOct 12 2021, 12:06 AM
Herald added a subscriber: Restricted Project. · View Herald Transcript
dvyukov accepted this revision.Oct 12 2021, 12:52 AM
dvyukov added inline comments.
compiler-rt/lib/sanitizer_common/sanitizer_stackdepot.cpp
28

nice

compiler-rt/lib/sanitizer_common/sanitizer_stackdepotbase.h
66

This effectively makes kReservedBits at least 1, so we could reduce kNodesSize1/2 by half.

This revision is now accepted and ready to land.Oct 12 2021, 12:52 AM
This revision was landed with ongoing or failed builds.Oct 12 2021, 3:53 PM
This revision was automatically updated to reflect the committed changes.