[PATCH] D142507: [AMDGPU] Split dot7 feature

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 24 14:55:53 PST 2023


rampitec added inline comments.


================
Comment at: clang/include/clang/Basic/BuiltinsAMDGPU.def:239
 
-TARGET_BUILTIN(__builtin_amdgcn_fdot2, "fV2hV2hfIb", "nc", "dot7-insts")
+TARGET_BUILTIN(__builtin_amdgcn_fdot2, "fV2hV2hfIb", "nc", "dot10-insts")
 TARGET_BUILTIN(__builtin_amdgcn_fdot2_f16_f16, "hV2hV2hh", "nc", "dot9-insts")
----------------
arsenm wrote:
> I have even less idea what these numbers mean now than I did before. This is also a bitcode compatibility break
They actually never meant anything just because there is no system in the support matrix. I know this one will need simultaneous update of the device lib downstream.


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

https://reviews.llvm.org/D142507



More information about the llvm-commits mailing list