[PATCH] D150375: [FuncSpec] Replace LoopInfo with BlockFrequencyInfo.

[Mikael Holmén via Phabricator via llvm-commits]

uabelho added a comment.

Both valgrind and ASan complains about memory problems. ASan says there is heap-use-after-free for stuff freed here

  #1 0x47b6bd9 in llvm::DomTreeUpdater::forceFlushDeletedBB()
  #2 0x47b649c in tryFlushDeletedBB
  #3 0x47b649c in llvm::DomTreeUpdater::dropOutOfDateUpdates()
  #4 0x174687e in llvm::DomTreeUpdater::~DomTreeUpdater()
  #5 0x8111719 in runIPSCCP(llvm::Module&, llvm::DataLayout const&, llvm::AnalysisManager<llvm::Function>*, std::__1::function<llvm::BlockFrequencyInfo& (llvm::Function&)>, std::__1::function<llvm::TargetLibraryInfo const& (llvm::Function&)>, std::__1::function<llvm::TargetTransformInfo& (llvm::Function&)>, std::__1::function<llvm::AssumptionCache& (llvm::Function&)>, llvm::function_ref<llvm::AnalysisResultsForFn (llvm::Function&)>, bool)

I've also seen crashes for other passes than adce, e.g. gvnhoist.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150375