[PATCH] D104575: AMDGPU: Try to eliminate clearing of high bits of 16-bit instructions

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 21 07:06:41 PDT 2021


arsenm added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUSubtarget.cpp:447
+    // changed to preserving the high bits.
+    return hasZeroHighBits16();
+  case AMDGPU::V_MAD_MIXLO_F16:
----------------
foad wrote:
> Where is this function defined?
Posted wrong version of patch. I added a subtarget feature for this initially, but the actual behavior is a bit too convoluted and would require multiple variants of it


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

https://reviews.llvm.org/D104575



More information about the llvm-commits mailing list