[PATCH] D24675: [AMDGPU] Stop using MCRegisterClass::getSize()

Krzysztof Parzyszek via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 16 18:06:12 PDT 2016


kparzysz added inline comments.

================
Comment at: lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.cpp:231-233
@@ +230,5 @@
+// (move from MC* level to Target* level). Return size in bits.
+unsigned getRegBitWidth(const MCRegisterClass &RC) {
+  switch (RC.getID()) {
+  case AMDGPU::SGPR_32RegClassID:
+  case AMDGPU::VGPR_32RegClassID:
----------------
tstellarAMD wrote:
> So will this function be replaced with a TargetRegisterInfo query in the future?  That's my impression from the description in the summary.
No. Unfortunately, there is no TargetRegisterInfo available at the places where this function is used.


Repository:
  rL LLVM

https://reviews.llvm.org/D24675





More information about the llvm-commits mailing list