Re: [PATCH v2 4/4] gdb/dwarf: use dynamic partitioning for DWARF CU indexing

[Simon Marchi <simon.marchi@polymtl.ca>]

On 9/18/25 3:53 PM, Tom Tromey wrote:
>>>>>> "Simon" == Simon Marchi <simon.marchi@efficios.com> writes:
> 
> Simon> Yeah, that's unfortunate.  It's because
> Simon> std::atomic<std::vector<...>::iterator> won't work with
> Simon> -D_GLIBCXX_DEBUG.  If I try to use iterators all the way, I get:
> 
> Simon> /opt/gcc/15.2.0/include/c++/15.2.0/atomic:214:21: error: static assertion failed: std::atomic requires a trivially copyable type
> Simon>   214 |       static_assert(__is_trivially_copyable(_Tp),
> Simon>       |                     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
> 
> Simon> Without _GLIBCXX_DEBUG, vector iterators boil down to just a pointer.
> Simon> But with _GLIBCXX_DEBUG, there is additional state.
> 
> Simon> I still want to be able to use _GLIBCXX_DEBUG, it's pretty useful.
> 
> Maybe a comment to this effect would be nice to have.
> 
> thanks,
> Tom

Ok, added.

Simon