wrotki accepted this revision. wrotki added a comment. Yes, it looks good. I downloaded the patch and ran the test - had some doubts about 'otool -L %t |grep -q "dynamic.dylib"' - but verified with other ASAN tests binaries and convinced myself that it's the right thing to do.
CHANGES SINCE LAST ACTION https://reviews.llvm.org/D156718/new/ https://reviews.llvm.org/D156718 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits