[PATCH] D128830: [Pipelines] Introduce DAE after ArgumentPromotion

[Martin Storsjö via Phabricator via cfe-commits]

mstorsjo added a comment.

This triggers failed asserts for me:

  $ cat repro.c 
  static float strtof(char *, char *) {}
  void a() { strtof(a, 0); }
  $ clang -target x86_64-w64-mingw32 -w -c repro.c -O3
  clang: ../lib/Analysis/CGSCCPassManager.cpp:958: updateCGAndAnalysisManagerForPass(llvm::LazyCallGraph&, llvm::LazyCallGraph::SCC&, llvm::LazyCallGraph::Node&, llvm::CGSCCAnalysisManager&, llvm::CGSCCUpdateResult&, llvm::FunctionAnalysisManager&, bool)::<lambda(llvm::Function&)>: Assertion `RefereeN && "Visited function should already have an associated node"' failed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128830