This is an archive of the discontinued LLVM Phabricator instance.

[dfsan] Introduce memory mapping for origin tracking
ClosedPublic

Authored by stephan.yichao.zhao on Feb 11 2021, 1:53 PM.

Diff Detail

Event Timeline

stephan.yichao.zhao requested review of this revision.Feb 11 2021, 1:53 PM
stephan.yichao.zhao created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptFeb 11 2021, 1:53 PM
Herald added a subscriber: Restricted Project. · View Herald Transcript
morehouse accepted this revision.Feb 11 2021, 2:08 PM
morehouse added inline comments.
compiler-rt/lib/dfsan/dfsan.h
84

const_cast

This revision is now accepted and ready to land.Feb 11 2021, 2:08 PM
stephan.yichao.zhao retitled this revision from [dfsan] Introduce memory mapping for origin tracking to [dfsan] Introduce memory mapping for origin tracking.

removed the unnecessary const_cast

stephan.yichao.zhao marked an inline comment as done.Feb 11 2021, 2:17 PM
stephan.yichao.zhao added inline comments.
compiler-rt/lib/dfsan/dfsan.h
84

changed the return value to be const dfsan_label*, so we do not need the cast.

This revision was landed with ongoing or failed builds.Feb 11 2021, 2:34 PM
This revision was automatically updated to reflect the committed changes.
dmajor added a subscriber: dmajor.Feb 11 2021, 7:42 PM

This commit broke the build on aarch64 because OriginAddr() isn't defined on that platform, could you please take a look?