[PATCH] D129492: Add missing sanitizer metadata plumbing from CFE.

2022-07-13 Thread Mitch Phillips via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG7045519359de: Add missing sanitizer metadata plumbing from CFE. (authored by hctim). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129492/new/ https://revie

[PATCH] D129492: Add missing sanitizer metadata plumbing from CFE.

2022-07-12 Thread Mitch Phillips via Phabricator via cfe-commits
hctim updated this revision to Diff 444115. hctim added a comment. Update ASan and HWASan tests as well. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129492/new/ https://reviews.llvm.org/D129492 Files: clang/lib/CodeGen/CodeGenModule.cpp clan

[PATCH] D129492: Add missing sanitizer metadata plumbing from CFE.

2022-07-12 Thread Evgenii Stepanov via Phabricator via cfe-commits
eugenis accepted this revision. eugenis added a comment. This revision is now accepted and ready to land. LGTM Is there an ASan test to update? I guess before this change, for an ignorelisted global, ASan would emit wrongly typed declaration on the undef side - which should not cause any practi

[PATCH] D129492: Add missing sanitizer metadata plumbing from CFE.

2022-07-11 Thread Mitch Phillips via Phabricator via cfe-commits
hctim created this revision. hctim added a reviewer: eugenis. Herald added a project: All. hctim requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. clang misses attaching sanitizer metadata for external globals. Repository: rG LLVM Github