summaryrefslogtreecommitdiff
path: root/lib/CodeGen/GlobalISel/RegisterBank.cpp
diff options
context:
space:
mode:
authorQuentin Colombet <qcolombet@apple.com>2016-04-05 22:34:01 +0000
committerQuentin Colombet <qcolombet@apple.com>2016-04-05 22:34:01 +0000
commit2b1f04632c6b390b816d69273ca041e039bb4e54 (patch)
tree2a0fca2e7883a18a153868b0b16edd9dbfcb3855 /lib/CodeGen/GlobalISel/RegisterBank.cpp
parent1f2bf92c75b4e63c8127b56f6bf35dd7841f137c (diff)
downloadllvm-2b1f04632c6b390b816d69273ca041e039bb4e54.tar.gz
[RegisterBank] Implement the verify method to check for the obvious mistakes.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@265479 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/CodeGen/GlobalISel/RegisterBank.cpp')
-rw-r--r--lib/CodeGen/GlobalISel/RegisterBank.cpp31
1 files changed, 27 insertions, 4 deletions
diff --git a/lib/CodeGen/GlobalISel/RegisterBank.cpp b/lib/CodeGen/GlobalISel/RegisterBank.cpp
index 9557bd0df78c..63a251297716 100644
--- a/lib/CodeGen/GlobalISel/RegisterBank.cpp
+++ b/lib/CodeGen/GlobalISel/RegisterBank.cpp
@@ -23,10 +23,33 @@ const unsigned RegisterBank::InvalidID = UINT_MAX;
RegisterBank::RegisterBank() : ID(InvalidID), Name(nullptr), Size(0) {}
void RegisterBank::verify(const TargetRegisterInfo &TRI) const {
- // Verify that the Size of the register bank is big enough to cover all the
- // register classes it covers.
- // Verify that the register bank covers all the sub and super classes of the
- // classes it covers.
+ assert(isValid() && "Invalid register bank");
+ assert(ContainedRegClasses.size() == TRI.getNumRegClasses() &&
+ "TRI does not match the initialization process?");
+ for (unsigned RCId = 0, End = TRI.getNumRegClasses(); RCId != End; ++RCId) {
+ const TargetRegisterClass &RC = *TRI.getRegClass(RCId);
+
+ if (!contains(RC))
+ continue;
+ // Verify that the register bank covers all the sub classes of the
+ // classes it covers.
+
+ // Use a different (slow in that case) method than
+ // RegisterBankInfo to find the subclasses of RC, to make sure
+ // both agree on the contains.
+ for (unsigned SubRCId = 0; SubRCId != End; ++SubRCId) {
+ const TargetRegisterClass &SubRC = *TRI.getRegClass(RCId);
+
+ if (!RC.hasSubClassEq(&SubRC))
+ continue;
+
+ // Verify that the Size of the register bank is big enough to cover
+ // all the register classes it covers.
+ assert((getSize() >= SubRC.getSize() * 8) &&
+ "Size is not big enough for all the subclasses!");
+ assert(contains(SubRC) && "Not all subclasses are covered");
+ }
+ }
}
bool RegisterBank::contains(const TargetRegisterClass &RC) const {