[PATCH] D80668: [Clang][Sanitizers] Expect test failure on {arm,thumb}v7

2020-05-30 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added a comment. `arc` adds many unneeded tags from Phabricator. You can drop `Reviewers:` `Subscribers:` `Tags:` and the text `Summary:` with the following script: arcfilter () { arc amend git log -1 --pretty=%B | awk '/Reviewers:|Subscribers:/{p=1} /Reviewed

[PATCH] D80668: [Clang][Sanitizers] Expect test failure on {arm,thumb}v7

2020-05-28 Thread Marco Elver via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG69935d86aed1: [Clang][Sanitizers] Expect test failure on {arm,thumb}v7 (authored by melver). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80668/new/

[PATCH] D80668: [Clang][Sanitizers] Expect test failure on {arm,thumb}v7

2020-05-28 Thread Marco Elver via Phabricator via cfe-commits
melver updated this revision to Diff 266773. melver added a comment. Add link to bug. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80668/new/ https://reviews.llvm.org/D80668 Files: clang/test/CodeGen/sanitize-coverage.c Index:

[PATCH] D80668: [Clang][Sanitizers] Expect test failure on {arm,thumb}v7

2020-05-28 Thread Alexander Potapenko via Phabricator via cfe-commits
glider added inline comments. Comment at: clang/test/CodeGen/sanitize-coverage.c:8 +// +// XFAIL: armv7, thumbv7 Is there a Bugzilla entry for this? Please add a link to the code and to the patch description. Repository: rG LLVM Github Monorepo CHANGES

[PATCH] D80668: [Clang][Sanitizers] Expect test failure on {arm,thumb}v7

2020-05-27 Thread Marco Elver via Phabricator via cfe-commits
melver added a comment. Example of failing test: http://lab.llvm.org:8011/builders/clang-cmake-armv7-full/builds/10689 So this definitely can't be -target, because we set that. So it seems that some version of LLVM compiled for armv7 is somehow broken? Vitaly, I wasn't able to test this since

[PATCH] D80668: [Clang][Sanitizers] Expect test failure on {arm,thumb}v7

2020-05-27 Thread Marco Elver via Phabricator via cfe-commits
melver created this revision. melver added a reviewer: vitalybuka. Herald added subscribers: cfe-commits, danielkiss, kristof.beyls. Herald added a project: clang. melver added a comment. Example of failing test: http://lab.llvm.org:8011/builders/clang-cmake-armv7-full/builds/10689 So this