[PATCH] D142990: [AMDGPU][NFC] More precise predicates on GFX9 f16 insts

Joe Nash via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 31 10:16:37 PST 2023


Joe_Nash added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/VOP3Instructions.td:306
 let FPDPRounding = 1 in {
   let Predicates = [Has16BitInsts, isGFX8Only] in {
     defm V_DIV_FIXUP_F16 : VOP3Inst <"v_div_fixup_f16", VOP3_Profile<VOP_F16_F16_F16_F16>, AMDGPUdiv_fixup>;
----------------
foad wrote:
> Can you make the same change here?
No. We explicitly want to override predicates on the pseudo instruction but not the real, so assembly of V_DIV_FIXUP_LEGACY_F16_gfx9 works. SubtargetPredicate would be copied to the real.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142990



More information about the llvm-commits mailing list