[PATCH] D156999: [Inline Spiller] Consider bundles when marking defs as dead

[Piotr Sobczak via Phabricator via llvm-commits]

piotr added inline comments.


================
Comment at: llvm/lib/CodeGen/InlineSpiller.cpp:768
+            LLVM_DEBUG(dbgs() << "All defs dead: " << *It);
+            DeadDefs.push_back(&*It);
+          }
----------------
arsenm wrote:
> My point was you should delete the whole bundle at a time, not just prune the copies out of it
Ok, I see. I thought copies are safe to just delete. Will make the change.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156999