On 2022-12-24 22:44, Tom Tromey wrote:
> PR symtab/29343 points out that it would be beneficial if
> comp_unit_head had a constructor and used initializers.  This patch
> implements this.

LGTM.

> I'm unsure if this is sufficient to close the bug,
> but at least it's a step.
> 

I think it's sufficient.

FWIW, I've rebased the patch attached to the PR on your patch, and I'm 
left with the patch below, which makes the length field of 
comp_unit_head private.

Thanks,
- Tom