[PATCH] Change how we deal with (implicit) -fno-rtti + -fsanitize=vptr
Filipe Cabecinhas
filcab+llvm.phabricator at gmail.com
Fri Feb 13 15:02:59 PST 2015
Changed it to retain current behavior (never enable rtti just because of a sanitizer), but fix the failing case (we were assuming -cc1 would care about -fno-sanitize=).
Refactored the code so the rtti/no-rtti decision was made in one place, and the other code that cares just asks the ToolChain.
http://reviews.llvm.org/D7525
Files:
include/clang/Driver/ToolChain.h
lib/Driver/SanitizerArgs.cpp
lib/Driver/ToolChain.cpp
lib/Driver/Tools.cpp
test/Driver/fsanitize.c
test/Driver/rtti-options.cpp
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D7525.19935.patch
Type: text/x-patch
Size: 13458 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150213/fda38e6b/attachment.bin>
More information about the cfe-commits
mailing list