[PATCH] D156529: [AMDGPU][True16] Pre-commit addition tests.

Fri Jul 28 06:42:42 PDT 2023

Joe_Nash added a comment.

In D156529#4542267 <https://reviews.llvm.org/D156529#4542267>, @arsenm wrote:

> Could we have a minimal 16-bit test for the bringup? One that tests all the basic 16-bit ops work on both  paths, but maybe doesn't bother trying to handle all the vector types and other complexities

Can you describe that a bit more? I would say the existing tests in tree have coverage of MC and Codegen for the fake16 and previous asic paths. And then it seems adding tests for the true16 path would mean a lot of them are xfail right now. But I do think more VOP2 true16 instructions than just add_f16 are functioning after the current patch stack. Perhaps we could add tests for those, such as mul_f16, sub_f16 etc.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156529