This is an archive of the discontinued LLVM Phabricator instance.

[memprof] Refactor out the MemInfoBlock into a macro based def.
ClosedPublic

Authored by snehasish on Jan 19 2022, 1:47 PM.

Details

Summary

This patch refactors out the MemInfoBlock definition into a macro based
header which can be included to generate enums, structus and code for
each field recorded by the memprof profiling runtime.

Diff Detail

Event Timeline

snehasish requested review of this revision.Jan 19 2022, 1:47 PM
snehasish created this revision.
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptJan 19 2022, 1:47 PM
Herald added subscribers: llvm-commits, Restricted Project. · View Herald Transcript
snehasish updated this revision to Diff 401407.Jan 19 2022, 2:32 PM

Rebase on formatting changes.

This revision is now accepted and ready to land.Jan 19 2022, 4:32 PM
This revision was landed with ongoing or failed builds.Feb 14 2022, 9:54 AM
This revision was automatically updated to reflect the committed changes.

Should be fixed with https://reviews.llvm.org/rGde54e4ab78ef09b60f870e8df6f8a87e56d6bd94. I will monitor for further issues. Thanks!

This seems to be a issue with c++20 modules build and not specific to LLDB. I've reverted this and the dependent commits since the fix will require more refactoring.