[PATCH] D16175: Introduce -fsanitize-stats flag.

2016-01-13 Thread Peter Collingbourne via cfe-commits
pcc created this revision. pcc added reviewers: kcc, eugenis. pcc added a subscriber: cfe-commits. This is part of a new statistics gathering feature for the sanitizers. See clang/docs/SanitizerStats.rst for further info and docs. http://reviews.llvm.org/D16175 Files: docs/SanitizerStats.rst

Re: [PATCH] D16175: Introduce -fsanitize-stats flag.

2016-01-14 Thread Ivan Krasin via cfe-commits
krasin1 added a subscriber: krasin1. krasin1 accepted this revision. krasin1 added a reviewer: krasin1. krasin1 added a comment. This revision is now accepted and ready to land. LGTM Comment at: lib/Driver/SanitizerArgs.cpp:560 @@ +559,3 @@ + LinkerOptionFlag = "--linker-option

Re: [PATCH] D16175: Introduce -fsanitize-stats flag.

2016-01-14 Thread Peter Collingbourne via cfe-commits
pcc updated this revision to Diff 44936. pcc added a comment. - Switch to an alternative in-memory representation that avoids the need for linker magic http://reviews.llvm.org/D16175 Files: docs/SanitizerStats.rst docs/UsersManual.rst docs/index.rst include/clang/AST/ASTConsumer.h in

Re: [PATCH] D16175: Introduce -fsanitize-stats flag.

2016-01-14 Thread Peter Collingbourne via cfe-commits
pcc updated this revision to Diff 44945. pcc marked an inline comment as done. pcc added a comment. - Add note about win32 function name mangling - Stop using -whole-archive to link the stats runtime http://reviews.llvm.org/D16175 Files: docs/SanitizerStats.rst docs/UsersManual.rst docs/i

Re: [PATCH] D16175: Introduce -fsanitize-stats flag.

2016-01-15 Thread Peter Collingbourne via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL257971: Introduce -fsanitize-stats flag. (authored by pcc). Changed prior to commit: http://reviews.llvm.org/D16175?vs=44945&id=45056#toc Repository: rL LLVM http://reviews.llvm.org/D16175 Files: