This is an archive of the discontinued LLVM Phabricator instance.

Revert "[ASan] Removed ASAN_SHADOW_SCALE."
ClosedPublic

Authored by kstoimenov on Dec 7 2021, 2:18 PM.

Diff Detail

Event Timeline

kstoimenov created this revision.Dec 7 2021, 2:18 PM
kstoimenov requested review of this revision.Dec 7 2021, 2:18 PM
Herald added a project: Restricted Project. · View Herald TranscriptDec 7 2021, 2:18 PM
Herald added a subscriber: Restricted Project. · View Herald Transcript
kstoimenov accepted this revision.Dec 7 2021, 2:19 PM
This revision is now accepted and ready to land.Dec 7 2021, 2:19 PM
This revision was landed with ongoing or failed builds.Dec 7 2021, 2:20 PM
This revision was automatically updated to reflect the committed changes.
MaskRay added a subscriber: MaskRay.Dec 7 2021, 3:14 PM

A revert needs a description why it was made so that people doing archaeology would have better idea what happened.
For this case they would be curious why the change was brought back.

Got it. Will provide revert description next time. In this case it was a build failure caused by bad merge.

Got it. Will provide revert description next time. In this case it was a build failure caused by bad merge.

Thanks:) Such reverts (to fix a build failure, you are the original author/reviewer, committed recently) don't need a revision. You could just push the commit :)