diff options
author | Eric Christopher <echristo@apple.com> | 2010-01-29 21:16:24 +0000 |
---|---|---|
committer | Eric Christopher <echristo@apple.com> | 2010-01-29 21:16:24 +0000 |
commit | 36664bfc7a540f1b45e80c966e36b1aea9d29aa3 (patch) | |
tree | 899942d3b3fd9c935e206176cabd10ab559c5e3c /lib/Transforms/Scalar | |
parent | e0638cb56c972f34931de2c69895bc0c54440447 (diff) | |
download | external_llvm-36664bfc7a540f1b45e80c966e36b1aea9d29aa3.zip external_llvm-36664bfc7a540f1b45e80c966e36b1aea9d29aa3.tar.gz external_llvm-36664bfc7a540f1b45e80c966e36b1aea9d29aa3.tar.bz2 |
Revert my last couple of patches. They appear to have broken bison.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@94841 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms/Scalar')
-rw-r--r-- | lib/Transforms/Scalar/SimplifyLibCalls.cpp | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/lib/Transforms/Scalar/SimplifyLibCalls.cpp b/lib/Transforms/Scalar/SimplifyLibCalls.cpp index fc23c42..a49da9c 100644 --- a/lib/Transforms/Scalar/SimplifyLibCalls.cpp +++ b/lib/Transforms/Scalar/SimplifyLibCalls.cpp @@ -1203,6 +1203,9 @@ struct MemMoveChkOpt : public LibCallOptimization { struct StrCpyChkOpt : public LibCallOptimization { virtual Value *CallOptimizer(Function *Callee, CallInst *CI, IRBuilder<> &B) { + // These optimizations require TargetData. + if (!TD) return 0; + const FunctionType *FT = Callee->getFunctionType(); if (FT->getNumParams() != 3 || FT->getReturnType() != FT->getParamType(0) || !isa<PointerType>(FT->getParamType(0)) || @@ -1214,13 +1217,8 @@ struct StrCpyChkOpt : public LibCallOptimization { if (!SizeCI) return 0; - // If a) we don't have any length information, or b) we know this will - // fit then just lower to a plain strcpy. Otherwise we'll keep our - // strcpy_chk call which may fail at runtime if the size is too long. - // TODO: It might be nice to get a maximum length out of the possible - // string lengths for varying. - if (SizeCI->isAllOnesValue() || - SizeCI->getZExtValue() >= GetStringLength(CI->getOperand(2))) + // We don't have any length information, just lower to a plain strcpy. + if (SizeCI->isAllOnesValue()) return EmitStrCpy(CI->getOperand(1), CI->getOperand(2), B); return 0; |