[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 =
----------------
sstefan1 wrote:
> xbolva00 wrote:
> > Check for overflow is missing
> If there is an overflow, this will not be executed.
Ok, great


Repository:
  rL LLVM

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

https://reviews.llvm.org/D65408