[PATCH] D103684: [AMDGPU] Stop using LegacyLegalizerInfo. NFCI.

Daniel Sanders via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 4 10:52:28 PDT 2021


dsanders accepted this revision.
dsanders added a comment.
This revision is now accepted and ready to land.

LGTM, Thanks!



================
Comment at: llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp:505
+  // s1 for VCC branches, s32 for SCC branches.
+  getActionDefinitionsBuilder(G_BRCOND).legalFor({S1, S32});
 
----------------
I'm not 100% sure off-hand if this one had any implicit clampScalar-like rules in the legacy info. I can believe that G_BRCOND only ever forms with s1/s32 though.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103684



More information about the llvm-commits mailing list