[PATCH] D79350: Don't check for inalloca in IPConstantPropagation

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 4 12:22:18 PDT 2020


aeubanks updated this revision to Diff 261893.
aeubanks added a comment.

Wrong version...


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D79350/new/

https://reviews.llvm.org/D79350

Files:
  llvm/lib/Transforms/IPO/IPConstantPropagation.cpp


Index: llvm/lib/Transforms/IPO/IPConstantPropagation.cpp
===================================================================
--- llvm/lib/Transforms/IPO/IPConstantPropagation.cpp
+++ llvm/lib/Transforms/IPO/IPConstantPropagation.cpp
@@ -129,7 +129,7 @@
   for (unsigned i = 0, e = ArgumentConstants.size(); i != e; ++i, ++AI) {
     // Do we have a constant argument?
     if (ArgumentConstants[i].getInt() || AI->use_empty() ||
-        AI->hasInAllocaAttr() || (AI->hasByValAttr() && !F.onlyReadsMemory()))
+        (AI->hasByValAttr() && !F.onlyReadsMemory()))
       continue;
 
     Value *V = ArgumentConstants[i].getPointer();


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D79350.261893.patch
Type: text/x-patch
Size: 634 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200504/4712c945/attachment.bin>


More information about the llvm-commits mailing list