[PATCH] D37867: [MSan] Add flag to disable use-after-dtor.

2017-09-14 Thread Matt Morehouse via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL313314: [MSan] Add flag to disable use-after-dtor. (authored by morehouse). Changed prior to commit: https://reviews.llvm.org/D37867?vs=115295=115317#toc Repository: rL LLVM

[PATCH] D37867: [MSan] Add flag to disable use-after-dtor.

2017-09-14 Thread Vitaly Buka via Phabricator via cfe-commits
vitalybuka accepted this revision. vitalybuka added inline comments. This revision is now accepted and ready to land. Comment at: clang/lib/Driver/SanitizerArgs.cpp:494 + options::OPT_fno_sanitize_memory_use_after_dtor, + false);

[PATCH] D37867: [MSan] Add flag to disable use-after-dtor.

2017-09-14 Thread Evgenii Stepanov via Phabricator via cfe-commits
eugenis accepted this revision. eugenis added a comment. Oh right, I was looking at https://reviews.llvm.org/D37860. https://reviews.llvm.org/D37867 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D37867: [MSan] Add flag to disable use-after-dtor.

2017-09-14 Thread Vitaly Buka via Phabricator via cfe-commits
vitalybuka added a comment. In https://reviews.llvm.org/D37867#871353, @eugenis wrote: > Have you looked at performance? Not the answer to your question, but this patch just adds negative flag without changing defaults. https://reviews.llvm.org/D37867

[PATCH] D37867: [MSan] Add flag to disable use-after-dtor.

2017-09-14 Thread Evgenii Stepanov via Phabricator via cfe-commits
eugenis added a comment. Have you looked at performance? https://reviews.llvm.org/D37867 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D37867: [MSan] Add flag to disable use-after-dtor.

2017-09-14 Thread Matt Morehouse via Phabricator via cfe-commits
morehouse updated this revision to Diff 115295. morehouse added a comment. - Use hasFlag() in CompilerInvocation.cpp as well. https://reviews.llvm.org/D37867 Files: clang/include/clang/Driver/Options.td clang/lib/Driver/SanitizerArgs.cpp clang/lib/Frontend/CompilerInvocation.cpp

[PATCH] D37867: [MSan] Add flag to disable use-after-dtor.

2017-09-14 Thread Matt Morehouse via Phabricator via cfe-commits
morehouse created this revision. Flag is -fno-sanitize-use-after-dtor. https://reviews.llvm.org/D37867 Files: clang/include/clang/Driver/Options.td clang/lib/Driver/SanitizerArgs.cpp clang/lib/Frontend/CompilerInvocation.cpp clang/test/Driver/fsanitize.c Index: