[PATCH] D83738: [Attributor]Introduce call base context argument pathway.

Kuter Dinel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 29 21:19:17 PDT 2020


kuter added inline comments.


================
Comment at: llvm/unittests/Transforms/IPO/AttributorTestBase.h:53
+    CallGraphUpdater CGUpdater;
+    BumpPtrAllocator Allocator;
+    InformationCache InfoCache(M, AG, Allocator, nullptr);
----------------
jdoerfert wrote:
> kuter wrote:
> > contents `BumpPtrAllocator` will be freed after this block is dead.
> > This causes a use after free. 
> Put the allocator in the same scope as the Attributor?
That's what I am planing to do. I just wanted to add a inline commend so I don't forget about this issue.



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83738



More information about the llvm-commits mailing list