[PATCH] D79628: [Clang][Driver] Add Bounds and Thread to SupportsCoverage list

2020-05-28 Thread Marco Elver via Phabricator via cfe-commits
melver added a comment. In D79628#2060254 , @alanphipps wrote: > clang/test/CodeGen/sanitize-coverage.c is also failing our downstream > embedded ARMv7 validations. https://reviews.llvm.org/D80668 has been submitted. It would be good if someone with

[PATCH] D79628: [Clang][Driver] Add Bounds and Thread to SupportsCoverage list

2020-05-28 Thread Alan Phipps via Phabricator via cfe-commits
alanphipps added a comment. clang/test/CodeGen/sanitize-coverage.c is also failing our downstream embedded ARMv7 validations. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D79628/new/ https://reviews.llvm.org/D79628

[PATCH] D79628: [Clang][Driver] Add Bounds and Thread to SupportsCoverage list

2020-05-27 Thread Marco Elver via Phabricator via cfe-commits
melver added a comment. In D79628#2058404 , @vvereschaka wrote: > the Clang sanitize-coverage.c test gets crashed on Windows platform for armv7 > target: > > - FAIL: Clang::sanitize-coverage.c > > ``` > - command stderr: Pass 'Run-time bounds checking'

[PATCH] D79628: [Clang][Driver] Add Bounds and Thread to SupportsCoverage list

2020-05-27 Thread Vlad Vereschaka via Phabricator via cfe-commits
vvereschaka added a comment. Hi Marco, the Clang sanitize-coverage.c test gets crashed on Windows platform for armv7 target: - FAIL: Clang::sanitize-coverage.c # command stderr: Pass 'Run-time bounds checking' is not initialized. Verify if there is a pass dependency cycle.

[PATCH] D79628: [Clang][Driver] Add Bounds and Thread to SupportsCoverage list

2020-05-26 Thread Vitaly Buka via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG14de6e29b131: [Clang][Driver] Add Bounds and Thread to SupportsCoverage list (authored by melver, committed by vitalybuka). Changed prior to commit: https://reviews.llvm.org/D79628?vs=266316=266324#toc

[PATCH] D79628: [Clang][Driver] Add Bounds and Thread to SupportsCoverage list

2020-05-26 Thread Vitaly Buka via Phabricator via cfe-commits
vitalybuka updated this revision to Diff 266316. vitalybuka added a comment. simplify some tests Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D79628/new/ https://reviews.llvm.org/D79628 Files: clang/lib/Driver/SanitizerArgs.cpp

[PATCH] D79628: [Clang][Driver] Add Bounds and Thread to SupportsCoverage list

2020-05-25 Thread Marco Elver via Phabricator via cfe-commits
melver updated this revision to Diff 266057. melver added a comment. Add missing -target to sanitize-coverage-bounds test. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D79628/new/ https://reviews.llvm.org/D79628 Files:

[PATCH] D79628: [Clang][Driver] Add Bounds and Thread to SupportsCoverage list

2020-05-25 Thread Marco Elver via Phabricator via cfe-commits
melver updated this revision to Diff 266054. melver added a comment. -triple -> -target Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D79628/new/ https://reviews.llvm.org/D79628 Files: clang/lib/Driver/SanitizerArgs.cpp

[PATCH] D79628: [Clang][Driver] Add Bounds and Thread to SupportsCoverage list

2020-05-25 Thread Marco Elver via Phabricator via cfe-commits
melver updated this revision to Diff 266048. melver added a comment. Herald added a project: Sanitizers. Herald added a subscriber: Sanitizers. Address failing tests: - Set -triple for test. - Some existing tests now pass under TSAN, and can be enabled. Repository: rG LLVM Github Monorepo

[PATCH] D79628: [Clang][Driver] Add Bounds and Thread to SupportsCoverage list

2020-05-14 Thread Marco Elver via Phabricator via cfe-commits
melver updated this revision to Diff 264112. melver added a comment. Fix formatting. PTAL. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D79628/new/ https://reviews.llvm.org/D79628 Files: clang/lib/Driver/SanitizerArgs.cpp

[PATCH] D79628: [Clang][Driver] Add Bounds and Thread to SupportsCoverage list

2020-05-14 Thread Marco Elver via Phabricator via cfe-commits
melver updated this revision to Diff 264109. melver added a comment. Add tests checking that when passing the combination of the sanitizer flags to Clang, we generate instrumentation for all enabled sanitizers. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D79628: [Clang][Driver] Add Bounds and Thread to SupportsCoverage list

2020-05-09 Thread Vitaly Buka via Phabricator via cfe-commits
vitalybuka added a comment. can you please add some basic IR test? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D79628/new/ https://reviews.llvm.org/D79628 ___ cfe-commits mailing list

[PATCH] D79628: [Clang][Driver] Add Bounds and Thread to SupportsCoverage list

2020-05-08 Thread Marco Elver via Phabricator via cfe-commits
melver created this revision. melver added a reviewer: vitalybuka. Herald added a project: clang. Herald added a subscriber: cfe-commits. This permits combining -fsanitize-coverage with -fsanitize=bounds or -fsanitize=thread. Note that, GCC already supports combining these. Bugzilla: