[PATCH] D76248: Fix a bug in the legacy inliner that causes subseqeunt double inlining.

Wenlei He via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 19 09:11:22 PDT 2020


wenlei added a comment.

Thanks for fixing this! I forgot to ask earlier, but this could happen with inlining under new PM as well, right? With new PM, we collect calls within SCC, then iterate over them for inlining, so for the same reason we could have dangling call in `VMap` from cloning?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76248





More information about the llvm-commits mailing list