[PATCH] D152905: [AMDGPU][GFX11] Add test coverage for 16-bit conversions, part 12.

Joe Nash via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 19 09:17:55 PDT 2023


Joe_Nash added inline comments.


================
Comment at: llvm/test/CodeGen/AMDGPU/function-args.ll:89
-; GCN-LABEL: {{^}}void_func_i16_zeroext:
-; GCN-NOT: v0
-; GCN: v_add_{{i|u}}32_e32 v0, {{(vcc, )?}}12, v0
----------------
The changes to llvm.amdgcn.raw.buffer.store.ll and llvm.amdgcn.struct.buffer.store.ll LGTM.

The changes to function-args.ll, I'm not so sure about. There are a lot of not lines. I don't know what properties these are ensuring, but I know something was being checked. When we remove those and go to auto-generated, there is no evidence some property was being checked. A use of v0 in this case.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152905



More information about the llvm-commits mailing list