[PATCH] D44645: [test] Fix Cross-DSO CFI Android sanitizer test for -rtlib=compiler-rt

Evgenii Stepanov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 19 16:00:13 PDT 2018


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 specific library name — but I don't know which library could 'accidentally' appear here)
Yes, it would be better to check for -NOT: libclang_rt.cfi


Repository:
  rC Clang

https://reviews.llvm.org/D44645





More information about the cfe-commits mailing list