ToolInvocation frees resources in the destructor but doesn't
have user-written copy c'tor or assignment operator, so copying it using
default ones can cause double free.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
LGTM, though I find the whole OwnsAction business to be a code smell. I think it'd be somewhat involved trying to address that though, so these changes are reasonable.