alekseyshl added a comment.
PTAL
https://reviews.llvm.org/D26461
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
alekseyshl updated this revision to Diff 77667.
alekseyshl added a comment.
- Remove unnecessary flag qualifications.
https://reviews.llvm.org/D26461
Files:
include/clang/Driver/Options.td
include/clang/Driver/SanitizerArgs.h
lib/Driver/SanitizerArgs.cpp
test/Driver/fsanitize.c
Index:
alekseyshl marked an inline comment as done.
alekseyshl added a comment.
.
https://reviews.llvm.org/D26461
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
alekseyshl updated this revision to Diff 77649.
alekseyshl added a comment.
- Add FIXME to address the rest of the suggestions later.
https://reviews.llvm.org/D26461
Files:
include/clang/Driver/Options.td
include/clang/Driver/SanitizerArgs.h
lib/Driver/SanitizerArgs.cpp
test/Driver/fsan
alekseyshl marked an inline comment as done.
alekseyshl added a comment.
Frontend options removed.
https://reviews.llvm.org/D26461
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
alekseyshl updated this revision to Diff 77647.
alekseyshl added a comment.
- Remove unnecessary frontend options.
https://reviews.llvm.org/D26461
Files:
include/clang/Driver/Options.td
include/clang/Driver/SanitizerArgs.h
lib/Driver/SanitizerArgs.cpp
test/Driver/fsanitize.c
Index: tes
alekseyshl updated this revision to Diff 77631.
alekseyshl added a comment.
- Translate new driver flags to legacy -llvm ones.
https://reviews.llvm.org/D26461
Files:
include/clang/Driver/Options.td
include/clang/Driver/SanitizerArgs.h
include/clang/Frontend/CodeGenOptions.def
lib/Driver
alekseyshl added inline comments.
Comment at: test/Driver/fsanitize.c:288
+// RUN: %clang -target x86_64-linux-gnu -fsanitize=thread
-fsanitize-thread-data-races -fno-sanitize-thread-data-races %s -### 2>&1 |
FileCheck %s --check-prefix=CHECK-TSAN-DATA-RACES-OFF
+// CHECK-TSAN-
alekseyshl updated this revision to Diff 77432.
alekseyshl marked 2 inline comments as done.
alekseyshl added a comment.
- Rename new options to better reflect their functionality.
https://reviews.llvm.org/D26461
Files:
include/clang/Driver/Options.td
include/clang/Driver/SanitizerArgs.h
alekseyshl updated this revision to Diff 77398.
alekseyshl added a comment.
- Added ust test for the new command line args.
https://reviews.llvm.org/D26461
Files:
include/clang/Driver/Options.td
include/clang/Driver/SanitizerArgs.h
include/clang/Frontend/CodeGenOptions.def
lib/CodeGen/B
alekseyshl added inline comments.
Comment at: lib/Frontend/CompilerInvocation.cpp:735
+ OPT_fno_sanitize_thread_data_races,
+ Opts.SanitizeThreadDataRaces);
+ Opts.SanitizeThreadStackTraces =
eugenis wrote:
> It seems common t
alekseyshl created this revision.
alekseyshl added a reviewer: eugenis.
alekseyshl added a subscriber: cfe-commits.
New clang flags, all default to true:
-f[no-]sanitize-thread-data-races
-f[no-]sanitize-thread-stack-traces
-f[no-]sanitize-thread-atomics
https://reviews.llvm.org/D26461
Files:
alekseyshl created this revision.
alekseyshl added a reviewer: eugenis.
alekseyshl added a subscriber: cfe-commits.
Use -fsanitize-recover instead of -mllvm -msan-keep-going: pass
-fsanitize-recover value to msan.
https://reviews.llvm.org/D26354
Files:
lib/CodeGen/BackendUtil.cpp
Index: li
13 matches
Mail list logo
