commit c3384995258efcec26acf3a88715dc99cd871c3e
author Torok Edwin <edwintorok@gmail.com> Thu Sep 24 18:33:42 2009 +0000
committer Torok Edwin <edwintorok@gmail.com> Thu Sep 24 18:33:42 2009 +0000
tree a1bfd4d5818635b817206fe5ee991673baab2901
parent 96b930ddc7f0df9e278a5cb65ad77a559a20964e

Constant propagating byval pointer is safe if function is readonly.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@82700 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Transforms/Scalar/SCCP.cpp

diff --git a/lib/Transforms/Scalar/SCCP.cpp b/lib/Transforms/Scalar/SCCP.cpp
index bf9ec0e..ad4bcc9 100644
--- a/lib/Transforms/Scalar/SCCP.cpp
+++ b/lib/Transforms/Scalar/SCCP.cpp
@@ -1267,7 +1267,7 @@
   for (Function::arg_iterator AI = F->arg_begin(), E = F->arg_end();
        AI != E; ++AI, ++CAI) {
     LatticeVal &IV = ValueState[AI];
-    if (AI->hasByValAttr() && isa<PointerType>(AI->getType())) {
+    if (AI->hasByValAttr() && !F->onlyReadsMemory()) {
       IV.markOverdefined();
       continue;
     }