Okay, yes there *is* a getKeyLength method on ValueName. And, it should be

used because we *do* want to allow nulls in names.

llvm-svn: 36178
This commit is contained in:
Reid Spencer 2007-04-16 22:01:57 +00:00
parent 6f3a548ff4
commit 9c9741e864

View File

@ -2823,7 +2823,7 @@ InstVal : ArithmeticOps Types ValueRef ',' ValueRef {
// Check for call to invalid intrinsic to avoid crashing later.
if (Function *theF = dyn_cast<Function>(V)) {
if (theF->hasName() && (strlen(theF->getValueName()->getKeyData()) >= 5)&&
if (theF->hasName() && (theF->getValueName()->getKeyLength() >= 5) &&
(0 == strncmp(theF->getValueName()->getKeyData(), "llvm.", 5)) &&
!theF->getIntrinsicID(true))
GEN_ERROR("Call to invalid LLVM intrinsic function '" +