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

[Joe Nash via Phabricator via llvm-commits]

Joe_Nash added inline comments.


================
Comment at: llvm/test/CodeGen/AMDGPU/v_mac.ll:9
-; GCN: buffer_load_dword [[C:v[0-9]+]], off, s[{{[0-9]+:[0-9]+}}], 0 offset:8
-; GCN: v_mac_f32_e32 [[C]], [[A]], [[B]]
-; GCN: buffer_store_dword [[C]]
----------------
The tests in these files appear to check for usage of mac or mad. So we would need to know whether equivalent fma/fmac should be expected on GFX11, and why they are not present. There are other tests for fma on GFX11, though I'm not sure if they cover similar cases. Additionally, the hand written tests basically look for a single instruction. It seems a shame to reduce that clarity. It might be better to skip this patch. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152909