[PATCH] D31874: AMDGPU: Switch aperture queries to use aperture registers instead of s_getreg_b32

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 10 11:03:40 PDT 2017


arsenm added inline comments.


================
Comment at: lib/Target/AMDGPU/SIRegisterInfo.cpp:1039-1040
 bool SIRegisterInfo::hasVGPRs(const TargetRegisterClass *RC) const {
+  if (!RC)
+    return false;
+
----------------
I would rather this assert on null than try to handle it


https://reviews.llvm.org/D31874





More information about the llvm-commits mailing list