Re: [PATCH] Document experimental status of odr

[Tom de Vries <tdevries@suse.de>]

On 2/26/21 2:04 PM, Mark Wielaard wrote:
> Hi Tom,
> 
> On Thu, Feb 25, 2021 at 05:17:31PM +0100, Tom de Vries wrote:
>> Add documentation of the experimental status of the odr optimization in dwz.1.
>>
>> Any comments?
>>
>>
>> 	PR dwz/27401
>> 	* dwz.1: Document experimental status of odr.
> 
> Looks good to me.
> 

Thanks for the review.

> BTW. Do you already know when would you consider it no longer
> experimental? Are there still fundemental bugs/design issues

With the current state of trunk + recently submitted odr fixes there are
no longer any known bugs in odr.

The only design issue that I can think of is that it's disabled for
low-mem (listed in set_die_odr_state as todo).

> or is it
> now just having people experiment with it and finding bugs?

That as well.

[ FWIW, I've unofficially labeled a recent commit as 0.14-rc1 and am in
the process of bringing that + submitted odr fixes to tumbleweed, with
odr enabled by default.  So, I expect some feedback there ... ;) ]

To come back to your question, the label experimental is to warn the
user that it can be used, but expect problems.  So I'd say once we fix
the initially reported problems and figure out the low-mem thingy, we
can drop the experimental label.

Thanks,
- Tom