[llvm] [llvm-exegesis] [AArch64] Resolving "not all operands are initialized by snippet generator" (PR #142529)

[Sjoerd Meijer via llvm-commits]

https://github.com/sjoerdmeijer commented:

I think this is forward progress as it now is able to successfully generate and execute more snippets and handle more cases, so looks good to me from that point of view, and I am okay that are limitations and TODOs.  However, before we merge this, I would like to see more crisps descriptions and comments, so that we understand better the limitations of the current work and what we would need to do next. 

The description of this patch isn't clear to me, it doesn't parse for me:

> Implementing randomizeTargetMCOperand() for AArch64 that omitting OPERAND_UNKNOWN and OPERAND_PCREL to an immediate value based on opcode.

I don't understand "omitting" here, so please clarify. The way I understand it at this moment, I think you mean to say that you now actually handle these operand types, but I could be wrong.



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