[PATCH] D65408: [Attributor] Heap-To-Stack Conversion

[Dávid Bolvanský via Phabricator via llvm-commits]

xbolva00 added inline comments.


================
Comment at: llvm/trunk/lib/Transforms/IPO/Attributor.cpp:3229
+        auto *SizeT = dyn_cast<ConstantInt>(MallocCall->getOperand(1));
+        APInt TotalSize = SizeT->getValue() * Num->getValue();
+        Size =
----------------
Check for overflow is missing


================
Comment at: llvm/trunk/lib/Transforms/IPO/Attributor.cpp:3263
+        Module *M = F->getParent();
+        Function *Fn = Intrinsic::getDeclaration(M, Intrinsic::memset, Tys);
+        CallInst::Create(Fn, Ops, "", BI->getNextNode());
----------------
Builder.CreateMemset


Repository:
  rL LLVM

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

https://reviews.llvm.org/D65408