[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