[PATCH] D141558: [MemProf] Collect access density statistics during profiling

2023-01-12 Thread Teresa Johnson via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGee73d240ab1d: [MemProf] Collect access density statistics during profiling (authored by tejohnson). Repository: rG LLVM Github Monorepo CHANGES

[PATCH] D141558: [MemProf] Collect access density statistics during profiling

2023-01-12 Thread Snehasish Kumar via Phabricator via cfe-commits
snehasish accepted this revision. snehasish added a comment. This revision is now accepted and ready to land. lgtm Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141558/new/ https://reviews.llvm.org/D141558

[PATCH] D141558: [MemProf] Collect access density statistics during profiling

2023-01-12 Thread Teresa Johnson via Phabricator via cfe-commits
tejohnson updated this revision to Diff 488766. tejohnson added a comment. Address comments (version change necessitated updating the tests) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141558/new/ https://reviews.llvm.org/D141558 Files:

[PATCH] D141558: [MemProf] Collect access density statistics during profiling

2023-01-12 Thread Snehasish Kumar via Phabricator via cfe-commits
snehasish added a comment. Thanks for updating the tests. Comment at: llvm/include/llvm/ProfileData/MemProfData.inc:35 // The version number of the raw binary format. #define MEMPROF_RAW_VERSION 1ULL Can you increment the version here? Then we won't

[PATCH] D141558: [MemProf] Collect access density statistics during profiling

2023-01-11 Thread Teresa Johnson via Phabricator via cfe-commits
tejohnson created this revision. tejohnson added reviewers: snehasish, davidxl. Herald added subscribers: Enna1, wenlei. Herald added a project: All. tejohnson requested review of this revision. Herald added projects: clang, Sanitizers, LLVM. Herald added subscribers: Sanitizers, cfe-commits.