This is an archive of the discontinued LLVM Phabricator instance.

[Docs][llvm-exegesis] Add documentation for memory
ClosedPublic

Authored by aidengrossman on May 20 2023, 4:16 PM.

Diff Detail

Event Timeline

aidengrossman created this revision.May 20 2023, 4:16 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 20 2023, 4:16 PM
Herald added a subscriber: mstojanovic. · View Herald Transcript
aidengrossman requested review of this revision.May 20 2023, 4:16 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 20 2023, 4:16 PM

@courbet Now that (most of) the memory annotation patch stack has landed, do you mind taking a look at this patch when you have time?

ondrasej added inline comments.Jul 3 2023, 5:26 AM
llvm/docs/CommandGuide/llvm-exegesis.rst
72

Typo: ...in bytes, the value is

79

Would it be possible to parse both decimal and hex? E.g. parse "123" as decimal, and "0x123" as hex.

Similar for the size of the memory (on the other side, the value stored in the memory might be easiest when kept only in hex).

aidengrossman marked an inline comment as done.

Address reviewer feedback.

llvm/docs/CommandGuide/llvm-exegesis.rst
79

Yes, this is possible, just not something I currently have implemented (but probably something I should write up a patch for at some point). Since this isn't supported currently, I'm thinking we leave the documentation as is, documenting the functionality currently, and then add in enhanced parsing along with documentation changes in a future patch. Thoughts?

gchatelet accepted this revision.Jul 18 2023, 5:43 AM
This revision is now accepted and ready to land.Jul 18 2023, 5:43 AM