[PATCH] D50285: [MC] Remove MCRegisterClass::getSize

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 3 17:04:40 PDT 2018


craig.topper created this revision.
craig.topper added a reviewer: kparzysz.

There is a comment that says this method is temporary to allow out of tree targets to migrate. It's been over a year and we just branched for 8.0. Is now a good time to remove it?


https://reviews.llvm.org/D50285

Files:
  include/llvm/MC/MCRegisterInfo.h


Index: include/llvm/MC/MCRegisterInfo.h
===================================================================
--- include/llvm/MC/MCRegisterInfo.h
+++ include/llvm/MC/MCRegisterInfo.h
@@ -82,8 +82,6 @@
 
   /// Return the size of the physical register in bytes.
   unsigned getPhysRegSize() const { return PhysRegSize; }
-  /// Temporary function to allow out-of-tree targets to switch.
-  unsigned getSize() const { return getPhysRegSize(); }
 
   /// getCopyCost - Return the cost of copying a value between two registers in
   /// this class. A negative number means the register class is very expensive


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D50285.159134.patch
Type: text/x-patch
Size: 605 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180804/89298dcb/attachment.bin>


More information about the llvm-commits mailing list