Page MenuHomePhabricator

Change deprecated -fsanitize-recover flag to apply to all sanitizers, not just UBSan.
ClosedPublic

Authored by rsmith on Apr 8 2020, 2:08 PM.

Details

Summary

This flag has been deprecated, with an on-by-default warning encouraging
users to explicitly specify whether they mean "all" or ubsan for 5 years
(released in Clang 3.7). Change it to mean what we wanted and
undeprecate it.

Also make the argument to -fsanitize-trap optional, and likewise default
it to 'all', and express the aliases for these flags in the .td file
rather than in code. (Plus documentation updates for the above.)

Diff Detail

Event Timeline

rsmith created this revision.Apr 8 2020, 2:08 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 8 2020, 2:08 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
vitalybuka accepted this revision.Apr 16 2020, 5:52 PM
This revision is now accepted and ready to land.Apr 16 2020, 5:52 PM
This revision was automatically updated to reflect the committed changes.