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

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 19 10:17:56 PDT 2020


davidxl added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/CloneFunction.cpp:361
+          // away later on.
+          if (!dyn_cast<CallBase>(NewInst) || !dyn_cast<CallBase>(V)) {
+            VMap[&*II] = V;
----------------
Is the first check needed (!dyn_cast<CallBase>(NewInst)) ?


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