[PATCH] Remove top-level Clang -fsanitize= flags for optional ASan features.

Alexey Samsonov vonosmas at gmail.com
Tue Jun 10 13:15:49 PDT 2014


Hi kcc,

init-order and use-after-return modes can currently be enabled
by runtime flags. use-after-scope mode is not really working at the
moment.

The only problem I see is that users won't be able to disable extra
instrumentation for init-order and use-after-scope by a top-level Clang flag.
But this instrumentation was implicitly enabled for quite a while and
we didn't hear from users hurt by it.

http://reviews.llvm.org/D4093

Files:
  include/llvm/Transforms/Instrumentation.h
  lib/Transforms/Instrumentation/AddressSanitizer.cpp
  projects/compiler-rt/test/asan/TestCases/initialization-blacklist.cc
  projects/compiler-rt/test/asan/TestCases/initialization-constexpr.cc
  projects/compiler-rt/test/asan/TestCases/initialization-nobug.cc
  projects/compiler-rt/test/asan/TestCases/use-after-scope-dtor-order.cc
  projects/compiler-rt/test/asan/TestCases/use-after-scope-inlined.cc
  projects/compiler-rt/test/asan/TestCases/use-after-scope-nobug.cc
  projects/compiler-rt/test/asan/TestCases/use-after-scope.cc
  test/Instrumentation/AddressSanitizer/debug_info.ll
  test/Instrumentation/AddressSanitizer/lifetime.ll
  test/Instrumentation/AddressSanitizer/stack-poisoning.ll
  tools/clang/docs/UsersManual.rst
  tools/clang/include/clang/Basic/Sanitizers.def
  tools/clang/lib/CodeGen/BackendUtil.cpp
  tools/clang/lib/CodeGen/CGDecl.cpp
  tools/clang/lib/Driver/SanitizerArgs.cpp
  tools/clang/test/CodeGen/sanitize-init-order.cpp
  tools/clang/test/CodeGen/sanitize-use-after-scope.c
  tools/clang/test/Driver/fsanitize.c
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D4093.10298.patch
Type: text/x-patch
Size: 25877 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140610/fec98f6b/attachment.bin>


More information about the llvm-commits mailing list