[PATCH] D132649: [AMDGPU][MC][GFX11][NFC] Add missing asm tests for VOP1 instructions

Thu Aug 25 07:46:45 PDT 2022

dp added a comment.

> A few things I am noticing: The tests seem to be split into 2 alphabetically sorted chunks. Can we just re-sort the file?

Sure, I'll re-sort the tests. The script appended missing tests to the end of the file which resulted in two sorted chunks.

Do you think we should keep existing tests if they are good enough? Or could we just replace them with generated tests?

> There appears to be no coverage for VOP1, VOP2, or VOPC promoted to the VOP3 form for the base instructions (we have coverage for vop3 dpp). Do you plan to cover that?

Yes, I noticed that. We also have very little coverage for VOPC.DPP, VOPCX.DPP and VOPD. I'm currently analyzing test coverage and will add missing tests for all encodings.

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

https://reviews.llvm.org/D132649