This is an archive of the discontinued LLVM Phabricator instance.

[clang] accept -fsanitize-ignorelist= in addition to -fsanitize-blacklist=
ClosedPublic

Authored by thakis on May 4 2021, 6:56 AM.

Details

Summary

Use that for internal names (including the default ignorelists of the
sanitizers).

Diff Detail

Event Timeline

thakis created this revision.May 4 2021, 6:56 AM
thakis requested review of this revision.May 4 2021, 6:56 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 4 2021, 6:56 AM
hans accepted this revision.May 4 2021, 7:20 AM

lgtm

clang/include/clang/Driver/Options.td
1524

Should the aliases above also have "Deprecated' in their help texts?

This revision is now accepted and ready to land.May 4 2021, 7:20 AM
This revision was landed with ongoing or failed builds.May 4 2021, 7:24 AM
This revision was automatically updated to reflect the committed changes.
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptMay 4 2021, 7:24 AM
Herald added a subscriber: Restricted Project. · View Herald Transcript

I think @vitalybuka has concrete suggestion on the naming.

I think @vitalybuka has concrete suggestion on the naming.

If ignorelist is used, I can delete -fsanitize-coverage-blocklist=

I think @vitalybuka has concrete suggestion on the naming.

If ignorelist is used, I can delete -fsanitize-coverage-blocklist=

lgtm

compiler-rt/test/asan/TestCases/initialization-ignorelist.cpp