[PATCH] D147776: [llvm-exegesis][AArch64] Handle OPERAND_UNKNOWN

[Clement Courbet via Phabricator via llvm-commits]

courbet added a comment.

I wanted gchatelet@ to have a look at this, because he wrote the `Instruction`/`Operand` code. In particular, I'm wondering why we're not using the info on the actual `Operand` value rather than relying on the `MCOperandInfo` for the operand. Specifically, we do know that this is a register since we return true for `isReg`, so we could just bake this knowledge into `getExplicitOperandInfo` in a target-independant way.

That being said, I don't know how the AArch64 backend is design, but it feels weird that the target info for that opearnd is `OPERAND_UNKNOWN` if we know that it's a register.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D147776/new/

https://reviews.llvm.org/D147776