[PATCH] D109300: [AMDGPU] Make vector super classes allocatable

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 7 16:56:35 PDT 2021


arsenm added a comment.

Can you specifically mention VGPRs and AGPRs in the commit message/description



================
Comment at: llvm/lib/Target/AMDGPU/SIISelLowering.cpp:11395
+          continue;
+        auto *RC = TRI->getRegClassForReg(MRI, Op.getReg());
+        if (!Op.getReg().isVirtual() || !TRI->hasAGPRs(RC))
----------------
Can you shortcut the getRegClassForReg if not virtual?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109300



More information about the llvm-commits mailing list