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&id=115317#toc
Repository:
rL LLVM
https://reviews.llvm.
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);
NeedP
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
http://lists.llvm.org/cgi-bin/mailman/listin
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
_
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
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
clang/t
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: clang/test/Driver/fsani
