[llvm] [AMDGPU] Add regbankselect rules for G_ADD/SUB and variants (PR #159860)
Petar Avramovic via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 26 02:15:55 PDT 2025
================
@@ -470,7 +470,16 @@ RegBankLegalizeRules::RegBankLegalizeRules(const GCNSubtarget &_ST,
.Uni(S16, {{Sgpr32Trunc}, {Sgpr32AExt, Sgpr32AExt}})
.Div(S16, {{Vgpr16}, {Vgpr16, Vgpr16}})
.Uni(S32, {{Sgpr32}, {Sgpr32, Sgpr32}})
- .Div(S32, {{Vgpr32}, {Vgpr32, Vgpr32}});
+ .Div(S32, {{Vgpr32}, {Vgpr32, Vgpr32}})
+ // Split 64-bit add/sub into two 32-bit ops on VGPRs
+ .Uni(S64, {{Vgpr64}, {Vgpr64, Vgpr64}, SplitTo32})
+ .Div(S64, {{Vgpr64}, {Vgpr64, Vgpr64}, SplitTo32})
+ .Uni(V2S16, {{SgprV2S16}, {SgprV2S16, SgprV2S16}})
+ .Div(V2S16, {{VgprV2S16}, {VgprV2S16, VgprV2S16}});
+
+ addRulesForGOpcs({G_UADDO, G_USUBO, G_UADDE, G_USUBE}, Standard)
+ .Uni(S32, {{Vgpr32, Vcc}, {Vgpr32, Vgpr32}})
----------------
petar-avramovic wrote:
yes,
also there are no tests for this (uniform)
no tests for G_UADDE and G_USUBE also, they have one more source operand (carry bit) and should use different rule
https://github.com/llvm/llvm-project/pull/159860
More information about the llvm-commits
mailing list