From d5c91b461a0fbd306afee8b85f81d5af95de59b7 Mon Sep 17 00:00:00 2001 From: Nick Lewycky Date: Sat, 3 Jan 2009 17:05:32 +0000 Subject: [PATCH] Any void readonly functions are provably dead, don't waste time adding nocapture attributes to them. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61610 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Transforms/IPO/FunctionAttrs.cpp | 14 -------------- 1 file changed, 14 deletions(-) diff --git a/lib/Transforms/IPO/FunctionAttrs.cpp b/lib/Transforms/IPO/FunctionAttrs.cpp index aea4cb4cf26..453b3c51aa2 100644 --- a/lib/Transforms/IPO/FunctionAttrs.cpp +++ b/lib/Transforms/IPO/FunctionAttrs.cpp @@ -307,20 +307,6 @@ bool FunctionAttrs::AddNoCaptureAttrs(const std::vector &SCC) { // External node - skip it; continue; - // If the function is readonly and doesn't return any value, we know that - // the pointer value is not captured. Mark all of its pointer arguments - // nocapture. - if (F->onlyReadsMemory() && F->getReturnType() == Type::VoidTy) { - for (Function::arg_iterator A = F->arg_begin(), E = F->arg_end(); - A != E; ++A) - if (isa(A->getType()) && !A->hasNoCaptureAttr()) { - A->addAttr(Attribute::NoCapture); - ++NumNoCapture; - Changed = true; - } - continue; - } - // Definitions with weak linkage may be overridden at linktime with // something that writes memory, so treat them like declarations. if (F->isDeclaration() || F->mayBeOverridden()) -- 2.11.0