[PATCH] D119878: [FuncSpec] Remove definitions of fully specialized functions.

[Alexandros Lamprineas via Phabricator via llvm-commits]

labrinea updated this revision to Diff 409392.
labrinea added a comment.

- Fixed a segfault which would happen if you tried to erase a function twice (replaced SmallVector with SmallPtrSet)
- Moved the deletion of dead instructions and functions in a destructor to make sure it's the last thing being called (though I am wondering if the Solver could still contain references to dead functions/instructions)
- Changed the dyn_cast_or_null to dyn_cast as suggested


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119878

Files:
  llvm/lib/Transforms/IPO/FunctionSpecialization.cpp
  llvm/test/Transforms/FunctionSpecialization/function-specialization-constant-expression3.ll
  llvm/test/Transforms/FunctionSpecialization/function-specialization.ll
  llvm/test/Transforms/FunctionSpecialization/function-specialization3.ll
  llvm/test/Transforms/FunctionSpecialization/function-specialization4.ll
  llvm/test/Transforms/FunctionSpecialization/remove-dead-recursive-function.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D119878.409392.patch
Type: text/x-patch
Size: 6940 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220216/a8e6f59a/attachment.bin>