[PATCH] D58589: [ConstantHoisting] Call releaseMemory() to avoid dangling elements in ConstIntInfoVec

Matthew Voss via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 25 10:45:31 PST 2019


ormris added inline comments.


================
Comment at: lib/Transforms/Scalar/ConstantHoisting.cpp:950
 
+  releaseMemory();
+
----------------
ormris wrote:
> If this only happens in the new pass manager, could this call be moved to ::run? That way, the legacy pass manager doesn't call this function twice.
s/::run/ConstantHoistingPass::run/g


Repository:
  rL LLVM

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

https://reviews.llvm.org/D58589





More information about the llvm-commits mailing list