[llvm] [llvm-mca] Add optional identifier field to mca::Instruction (PR #97867)

Aiden Grossman via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 11 11:26:31 PDT 2024


boomanaiden154 wrote:

> Commenting for bump.

Can you reply to Michael's questions above? I'd be interest to know why the listed alternatives don't work.

The change seems fine to me. It does increase the size of `mca::Instruction`, but given it's already heap allocated and large, that's probably not too big of an issue.

https://github.com/llvm/llvm-project/pull/97867


More information about the llvm-commits mailing list