[llvm] [llvm-mca] Add optional identifier field to mca::Instruction (PR #97867)
Aiden Grossman via llvm-commits
llvm-commits at lists.llvm.org
Sun Jul 14 23:54:02 PDT 2024
boomanaiden154 wrote:
Adding an identifier seems like it might not be a bad option given the current design. If it helps multiple downstream consumers, then it seems reasonable enough to me. We don't need any of this sort of functionality in our downstream usage (so far), but I can see how it would potentially be useful for others.
I'll defer to the primary `llvm-mca` reviewers here as I'm not really qualified to review changes like this in this area of the project.
https://github.com/llvm/llvm-project/pull/97867
More information about the llvm-commits
mailing list