eugenis added a comment.
This was fixed in https://reviews.llvm.org/D44655
https://reviews.llvm.org/D44645
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mgorny marked 2 inline comments as done.
mgorny added a comment.
@eugenis, updated.
https://reviews.llvm.org/D44645
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mgorny updated this revision to Diff 139135.
mgorny edited the summary of this revision.
https://reviews.llvm.org/D44645
Files:
test/Driver/sanitizer-ld.c
Index: test/Driver/sanitizer-ld.c
===
--- test/Driver/sanitizer-ld.c
+++
eugenis added inline comments.
Comment at: test/Driver/sanitizer-ld.c:517
// CHECK-CFI-CROSS-DSO-ANDROID: "{{.*}}ld{{(.exe)?}}"
// CHECK-CFI-CROSS-DSO-ANDROID-NOT: libclang_rt.
mgorny wrote:
> (an alternative would be to replace this 'NOT' clause with more
mgorny added inline comments.
Comment at: test/Driver/sanitizer-ld.c:517
// CHECK-CFI-CROSS-DSO-ANDROID: "{{.*}}ld{{(.exe)?}}"
// CHECK-CFI-CROSS-DSO-ANDROID-NOT: libclang_rt.
(an alternative would be to replace this 'NOT' clause with more specific
library
mgorny created this revision.
mgorny added a reviewer: eugenis.
Herald added subscribers: dberris, srhines.
mgorny added inline comments.
Comment at: test/Driver/sanitizer-ld.c:517
// CHECK-CFI-CROSS-DSO-ANDROID: "{{.*}}ld{{(.exe)?}}"
// CHECK-CFI-CROSS-DSO-ANDROID-NOT: