[PR debug/47590] rework md option overriding to delay var-tracking

Alexandre Oliva aoliva@redhat.com
Wed Jun 1 22:51:00 GMT 2011


On Jun  1, 2011, Bernd Schmidt <bernds@codesourcery.com> wrote:

> On 06/01/2011 10:10 PM, Alexandre Oliva wrote:
>> On May  4, 2011, Bernd Schmidt <bernds@codesourcery.com> wrote:
>> 
>>> This comment looks very weird when added to ia64_option_override
>>> (likewise for other targets). Is there a reason it's not true anymore?
>> 
>> Dunno, but the patch definitely didn't work any more when I retested it.
>> Maybe it didin't work when I first tested it, I don't recall having
>> actually looked at debug dumps then, but it was a while ago.
>> 
>> Here's an alternate approach that works now.  Regstrapped on
>> 32- and 64-bit x86-linux-gnu, and cross-built on x86_64-linux-gnu to the
>> 4 affected platforms, checking manually the proper presence of debug
>> insns and location notes at the expected dump files.
>> 
>> Ok to install?

> Looks ok, except I think you need to update tm.texi.in and tm.texi?

Oh, I didn't realize updating tm.texi.in; AFAICT tm.texi is generated
the same regardless.  Anyway, here's a patch with reordered hunks, a
proper ChangeLog (I failed to update it before posting) and with the
changes for tm.texi propagated to tm.texi.in.

I'll check it in soon if there aren't objections.

Thanks,

-------------- next part --------------
A non-text attachment was scrubbed...
Name: delay-var-tracking-options-pr47590.patch
Type: text/x-diff
Size: 18167 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20110601/ce37364a/attachment.bin>
-------------- next part --------------


-- 
Alexandre Oliva, freedom fighter    http://FSFLA.org/~lxoliva/
You must be the change you wish to see in the world. -- Gandhi
Be Free! -- http://FSFLA.org/   FSF Latin America board member
Free Software Evangelist      Red Hat Brazil Compiler Engineer


More information about the Gcc-patches mailing list