This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: RFA 0/2 fix for bug with large structure offsets
- From: Pedro Alves <palves at redhat dot com>
- To: Yao Qi <yao at codesourcery dot com>, David Taylor <dtaylor at usendtaylorx2l dot lss dot emc dot com>
- Cc: "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>
- Date: Fri, 12 Dec 2014 11:15:36 +0000
- Subject: Re: RFA 0/2 fix for bug with large structure offsets
- Authentication-results: sourceware.org; auth=none
- References: <18829 dot 1416329565 at usendtaylorx2l> <23959 dot 1418329569 at usendtaylorx2l> <87zjaty0ze dot fsf at codesourcery dot com>
On 12/12/2014 03:56 AM, Yao Qi wrote:
> David Taylor <dtaylor@usendtaylorx2l.lss.emc.com> writes:
>
>> In the slightly over three weeks since I posted this and the two related
>> parts, I have seen only two replies off list, nothing on list.
>>
>> This bug is affecting us and I'd like some version of a fix for it to go
>> in. Any chance?
>
> It is worried that we can't verify the type width change like this.
Hmm. Is it really a big problem though? What's the worse failure mode if we
miss converting some code path? It seems that even if that happens, we won't
be worse than today?
> Personally, I don't see the point GDB-switch-to-C++ block the changes,
> either from your patch or from Fedora GDB's patches, although I agree
> that it is easier to do sanity check in C++.
Agreed.
> If it is practical to turn on -Wconversion and people here agree to do
> so, we can do it first to pave the way for your patch, and convince
> people your patch is safe enough. Last time I tried turning
> -Wconversion on, it causes a huge number of warnings. :-/
I'm not even sure it makes sense to have -Wconversion (or its variants)
block this?
If we agree on the types to use, then it seems to me it's more
useful to push the patches in than hold them up.
Thanks,
Pedro Alves