From: Michael Gottesman Date: Wed, 3 Jul 2013 04:00:54 +0000 (+0000) Subject: Added support in FunctionAttrs for adding relevant function/argument attributes for... X-Git-Url: http://plrg.eecs.uci.edu/git/?a=commitdiff_plain;h=7cb0321ad8d86c9fdfb70d3bfa1ff3e8e1ff9f71;p=oota-llvm.git Added support in FunctionAttrs for adding relevant function/argument attributes for the posix call gettimeofday. This implies annotating it as nounwind and its arguments as nocapture. To be conservative, we do not annotate the arguments with noalias since some platforms do not have restrict on the declaration for gettimeofday. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@185502 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Transforms/IPO/FunctionAttrs.cpp b/lib/Transforms/IPO/FunctionAttrs.cpp index 79ce3c33057..7df556ebffa 100644 --- a/lib/Transforms/IPO/FunctionAttrs.cpp +++ b/lib/Transforms/IPO/FunctionAttrs.cpp @@ -1310,6 +1310,16 @@ bool FunctionAttrs::inferPrototypeAttributes(Function &F) { // May throw; "open" is a valid pthread cancellation point. setDoesNotCapture(F, 1); break; + case LibFunc::gettimeofday: + if (FTy->getNumParams() != 2 || !FTy->getParamType(0)->isPointerTy() || + !FTy->getParamType(1)->isPointerTy()) + return false; + // Currently some platforms have the restrict keyword on the arguments to + // gettimeofday. To be conservative, do not add noalias to gettimeofday's + // arguments. + setDoesNotThrow(F); + setDoesNotCapture(F, 1); + setDoesNotCapture(F, 2); default: // Didn't mark any attributes. return false; diff --git a/test/Transforms/FunctionAttrs/annotate-1.ll b/test/Transforms/FunctionAttrs/annotate-1.ll index ae77380acc4..d8e90cf1c38 100644 --- a/test/Transforms/FunctionAttrs/annotate-1.ll +++ b/test/Transforms/FunctionAttrs/annotate-1.ll @@ -14,5 +14,8 @@ declare i32* @realloc(i32*, i32) declare i32 @strcpy(...) ; CHECK: declare i32 @strcpy(...) +declare i32 @gettimeofday(i8*, i8*) +; CHECK: declare i32 @gettimeofday(i8* nocapture, i8* nocapture) [[G0]] + ; CHECK: attributes [[G0]] = { nounwind } ; CHECK: attributes [[G1]] = { nounwind readonly }