From: Chris Lattner Date: Sat, 7 Nov 2009 18:53:00 +0000 (+0000) Subject: all targets should be required to declare legal integer types. My plan to X-Git-Url: http://plrg.eecs.uci.edu/git/?a=commitdiff_plain;h=9e496d223102cb3d5e37c513a697e30532e71bc0;p=oota-llvm.git all targets should be required to declare legal integer types. My plan to make it optional doesn't work out. If you don't want to specify this, don't specify a TD string at all. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@86394 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/include/llvm/Target/TargetData.h b/include/llvm/Target/TargetData.h index 96308778582..b663d4792ba 100644 --- a/include/llvm/Target/TargetData.h +++ b/include/llvm/Target/TargetData.h @@ -144,23 +144,22 @@ public: /// string constructor above. std::string getStringRepresentation() const; - - /// isIllegalInteger - This function returns true if the specified type is - /// known to not be a native integer type supported by the CPU. For example, + /// isLegalInteger - This function returns true if the specified type is + /// known tobe a native integer type supported by the CPU. For example, /// i64 is not native on most 32-bit CPUs and i37 is not native on any known - /// one. This returns false if the integer width is legal or we don't know. + /// one. This returns false if the integer width is not legal. /// /// The width is specified in bits. /// - bool isIllegalInteger(unsigned Width) const { - // If we don't have information about legal integer types, don't claim the - // type is illegal. - if (LegalIntWidths.empty()) return false; - + bool isLegalInteger(unsigned Width) const { for (unsigned i = 0, e = LegalIntWidths.size(); i != e; ++i) if (LegalIntWidths[i] == Width) - return false; - return true; + return true; + return false; + } + + bool isIllegalInteger(unsigned Width) const { + return !isLegalInteger(Width); } /// Target pointer alignment