This is an archive of the discontinued LLVM Phabricator instance.

[sanitizer] Add Lempel–Ziv–Welch encoder/decoder
ClosedPublic

Authored by vitalybuka on Dec 1 2021, 4:51 PM.

Details

Summary

It's very simple, fast and efficient for the stack depot compression if used on entire pointers.

Diff Detail

Event Timeline

vitalybuka created this revision.Dec 1 2021, 4:51 PM
vitalybuka requested review of this revision.Dec 1 2021, 4:51 PM
Herald added a project: Restricted Project. · View Herald TranscriptDec 1 2021, 4:51 PM
Herald added a subscriber: Restricted Project. · View Herald Transcript
kstoimenov accepted this revision.Dec 2 2021, 8:34 AM
kstoimenov added inline comments.
compiler-rt/lib/sanitizer_common/sanitizer_lzw.h
36

Why not dict?

This revision is now accepted and ready to land.Dec 2 2021, 8:34 AM
vitalybuka added inline comments.Dec 2 2021, 10:13 AM
compiler-rt/lib/sanitizer_common/sanitizer_lzw.h
36

to show that all stuff there is len 1

kstoimenov added inline comments.Dec 2 2021, 10:20 AM
compiler-rt/lib/sanitizer_common/sanitizer_lzw.h
36

When I see dict1 I expect to see dict2. Maybe rename to dict_len1 or dict_l1 to make it more obvious?

vitalybuka marked an inline comment as done.

update

morehouse accepted this revision.Dec 2 2021, 2:09 PM
morehouse added inline comments.
compiler-rt/lib/sanitizer_common/tests/sanitizer_lzw_test.cpp
47

Delete this?

83
vitalybuka updated this revision to Diff 391476.Dec 2 2021, 3:14 PM
vitalybuka marked 2 inline comments as done.

fixes

morehouse accepted this revision.Dec 2 2021, 3:15 PM
This revision was automatically updated to reflect the committed changes.