This is an archive of the discontinued LLVM Phabricator instance.

[ASAN] Remove asserts introduced in https://reviews.llvm.org/D136197
ClosedPublic

Authored by rsundahl on Oct 24 2022, 4:05 PM.

Details

Summary

Additional calls were introduced for outlining (opposite of inlining)
in https://reviews.llvm.org/D136197 which contain asserts that partial
poisoning of a single byte wouldn't happen consecutively but this is
too strong and actually does occur in Windows. Removing those asserts
as they are unnecessary

Diff Detail

Event Timeline

rsundahl created this revision.Oct 24 2022, 4:05 PM
Herald added a project: Restricted Project. · View Herald TranscriptOct 24 2022, 4:05 PM
Herald added a subscriber: Enna1. · View Herald Transcript
rsundahl requested review of this revision.Oct 24 2022, 4:05 PM
Herald added a project: Restricted Project. · View Herald TranscriptOct 24 2022, 4:05 PM
Herald added a subscriber: Restricted Project. · View Herald Transcript
vitalybuka accepted this revision.Oct 24 2022, 4:36 PM
This revision is now accepted and ready to land.Oct 24 2022, 4:36 PM