It'd be good if this could go in today. ubsanvptr has been broken for us for a while now.
On Mon, Jul 31, 2017 at 4:43 PM, Vedant Kumar via Phabricator via cfe-commits <cfe-commits@lists.llvm.org> wrote: > vsk created this revision. > > In r309007, I made -fsanitize=null a hard prerequisite for > -fsanitize=vptr. I did not see the need for the two checks to have separate > null checking logic for the same pointer. I expected the two checks to > either always be enabled together, or to be mutually compatible. > > In the mailing list discussion re: r309007 it became clear that that isn't > the case. If a codebase is -fsanitize=vptr clean but not -fsanitize=null > clean, it's useful to have -fsanitize=vptr emit its own null check. That's > what this patch does: with it, -fsanitize=vptr can be used without > -fsanitize=null. > > > https://reviews.llvm.org/D36112 > > Files: > docs/ReleaseNotes.rst > docs/UndefinedBehaviorSanitizer.rst > include/clang/Basic/DiagnosticDriverKinds.td > lib/CodeGen/CGExpr.cpp > lib/Driver/SanitizerArgs.cpp > test/CodeGenCXX/catch-undef-behavior.cpp > test/CodeGenCXX/ubsan-type-checks.cpp > test/Driver/fsanitize.c > test/Driver/rtti-options.cpp > > > _______________________________________________ > cfe-commits mailing list > cfe-commits@lists.llvm.org > http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits > >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
