This is an archive of the discontinued LLVM Phabricator instance.

Remove esan.
ClosedPublic

Authored by thakis on Mar 8 2019, 6:43 AM.

Details

Summary

It hasn't seen active development in years, and it hasn't reached a state where it was useful.

Remove the code until someone is interested in working on it again.

Diff Detail

Repository
rC Clang

Event Timeline

thakis created this revision.Mar 8 2019, 6:43 AM
Herald added a project: Restricted Project. · View Herald Transcript

How about DFSan and shadowcallstack?

How about DFSan and shadowcallstack?

I don't understand how that question is related to this change. What do you mean?

How about DFSan and shadowcallstack?

I don't understand how that question is related to this change. What do you mean?

shadowcallstack still misses even runtime.

DFSan seems to be unmaintained.

Are they planned to be dropped?

shadowcallstack still misses even runtime.

See https://reviews.llvm.org/D59034 for shadowcallstack.

eugenis accepted this revision.Mar 8 2019, 10:49 AM
eugenis added a subscriber: pcc.

LGTM, good job!

Regarding dfsan, I think it has users @pcc

This revision is now accepted and ready to land.Mar 8 2019, 10:49 AM
pcc added a comment.Mar 8 2019, 10:57 AM

LGTM, good job!

Regarding dfsan, I think it has users @pcc

Yes, it has users, and LibFuzzer has a mode which uses it.

pcc added a comment.Mar 8 2019, 10:59 AM

How about DFSan and shadowcallstack?

I don't understand how that question is related to this change. What do you mean?

shadowcallstack still misses even runtime.

The intent is that the runtime is supplied by the operating system or application. See https://clang.llvm.org/docs/ShadowCallStack.html for a description of the Android runtime.

LGTM, good job!

Regarding dfsan, I think it has users @pcc

It was in plans to be ported to NetBSD.. but we wondered if there are any users.

This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptMar 11 2019, 1:22 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
test/Driver/sanitizer-ld.c