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 L
wrotki added a comment.
LGTM, perhaps with addressing @RKSimon remark about release notes
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D145715/new/
https://reviews.llvm.org/D145715
___
cfe-commits mailin