[commit, spu] Fix regression (Re: [PR debug/47590] rework md option overriding to delay var-tracking)

2011-07-13 Thread Ulrich Weigand
Alexandre Oliva wrote: * config/spu/spu.c (spu_flag_var_tracking): Drop. (TARGET_DELAY_VARTRACK): Define. (spu_var_tracking): New. (spu_machine_dependent_reorg): Call it. (asm_file_start): Don't save and override flag_var_tracking. This change caused crashes

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

2011-06-03 Thread Alexandre Oliva
On Jun 2, 2011, Alexandre Oliva aol...@redhat.com wrote: Right you are, though it looks like leaving the @hook lines out makes no difference. Anyhow, here's the patch I'm checking in. Trunk is now fixed, is the patch ok for the 4.6 branch too? for gcc/ChangeLog from Alexandre Oliva

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

2011-06-03 Thread Bernd Schmidt
On 06/03/2011 04:11 PM, Alexandre Oliva wrote: Trunk is now fixed, is the patch ok for the 4.6 branch too? As far as I'm concerned, sure. Bernd

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

2011-06-02 Thread Alexandre Oliva
On Jun 1, 2011, Bernd Schmidt ber...@codesourcery.com wrote: On 06/02/2011 12:47 AM, Alexandre Oliva wrote: On Jun 1, 2011, Bernd Schmidt ber...@codesourcery.com wrote: 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

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

2011-06-01 Thread Alexandre Oliva
On May 4, 2011, Bernd Schmidt ber...@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

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

2011-06-01 Thread Bernd Schmidt
On 06/01/2011 10:10 PM, Alexandre Oliva wrote: On May 4, 2011, Bernd Schmidt ber...@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

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

2011-06-01 Thread Alexandre Oliva
On Jun 1, 2011, Bernd Schmidt ber...@codesourcery.com wrote: On 06/01/2011 10:10 PM, Alexandre Oliva wrote: On May 4, 2011, Bernd Schmidt ber...@codesourcery.com wrote: This comment looks very weird when added to ia64_option_override (likewise for other targets). Is there a reason it's

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

2011-05-04 Thread Bernd Schmidt
On 04/02/2011 10:15 AM, Alexandre Oliva wrote: Some targets delayed the var-tracking pass to run it after machine-specific transformations. The introduction of option saving and restoring broke this, because the machine-specific overriding took place too late for it to be saved, so, after