[PATCH] D155006: [YAML][NFC] Use BumpPtrAllocator instead of unique_ptrs

[Amir Ayupov via Phabricator via llvm-commits]

Amir added a comment.

In D155006#4573881 <https://reviews.llvm.org/D155006#4573881>, @dblaikie wrote:

> What are the codepaths that currently destroy these nodes?

>From what I can tell there are just two: destructor of Input class and reassignment of TopNode in setCurrentDocument.

> Is there any mutability in the data structure, or is it only "build build build, use, then destroy"? If it's the latter, then yeah, arena allocation seems fine.

I didn't observe any mutability, it's the latter.

>> It's possible to match unique_ptr behavior and also free memory in setCurrentDocument.
>
> Ah, so perhaps this answers my question - the deallocation is done at `TopNode = `? *nod* Wuold probably be good to match the behavior there, to avoid continuous growth.

Will do.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155006