diff options
author | Quentin Colombet <qcolombet@apple.com> | 2016-04-05 22:34:01 +0000 |
---|---|---|
committer | Quentin Colombet <qcolombet@apple.com> | 2016-04-05 22:34:01 +0000 |
commit | 2b1f04632c6b390b816d69273ca041e039bb4e54 (patch) | |
tree | 2a0fca2e7883a18a153868b0b16edd9dbfcb3855 /lib/CodeGen/GlobalISel | |
parent | 1f2bf92c75b4e63c8127b56f6bf35dd7841f137c (diff) | |
download | llvm-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')
-rw-r--r-- | lib/CodeGen/GlobalISel/RegisterBank.cpp | 31 |
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 { |