[all-commits] [llvm/llvm-project] c8248d: Change deprecated -fsanitize-recover flag to apply...

Richard Smith via All-commits all-commits at lists.llvm.org
Fri Apr 17 22:48:54 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: c8248dc3bb36bea61ac6d87bd02c39c6a781b011
      https://github.com/llvm/llvm-project/commit/c8248dc3bb36bea61ac6d87bd02c39c6a781b011
  Author: Richard Smith <richard at metafoo.co.uk>
  Date:   2020-04-17 (Fri, 17 Apr 2020)

  Changed paths:
    M clang/docs/ReleaseNotes.rst
    M clang/docs/UndefinedBehaviorSanitizer.rst
    M clang/docs/UsersManual.rst
    M clang/include/clang/Driver/Options.td
    M clang/lib/Driver/SanitizerArgs.cpp
    M clang/test/Driver/fsanitize.c

  Log Message:
  -----------
  Change deprecated -fsanitize-recover flag to apply to all sanitizers, not just UBSan.

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.)

Reviewers: kcc

Subscribers: cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D77753




More information about the All-commits mailing list