Details
Diff Detail
Event Timeline
@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?
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). |
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? |
Typo: ...in bytes, the value is