RE: try_merge_delay_insn with delay list > 1

2015-04-21 Thread BELBACHIR Selim
Great, I'll read more closely formatting rules next time I'll submit something. Regards, Selim -Message d'origine- De : Jeff Law [mailto:l...@redhat.com] Envoyé : lundi 20 avril 2015 19:47 À : BELBACHIR Selim; gcc@gcc.gnu.org Objet : Re: try_merge_delay_insn with delay

Re: try_merge_delay_insn with delay list > 1

2015-04-20 Thread Jeff Law
On 04/20/2015 05:08 AM, BELBACHIR Selim wrote: I've attached the fixed version of the patch. I've tested it on the trunk with my private target. I can't provide a test because apparently no backend (other than my private one) uses delay slots with more that 1 slot. I was also unable to test th

RE: try_merge_delay_insn with delay list > 1

2015-04-20 Thread BELBACHIR Selim
t.com] Envoyé : vendredi 17 avril 2015 18:41 À : BELBACHIR Selim; gcc@gcc.gnu.org Objet : Re: try_merge_delay_insn with delay list > 1 On 03/10/2015 07:40 AM, BELBACHIR Selim wrote: > Me again :) > > I enhanced my patch because it was not generalized for instructions with N > delay

Re: try_merge_delay_insn with delay list > 1

2015-04-17 Thread Jeff Law
On 03/10/2015 07:40 AM, BELBACHIR Selim wrote: Me again :) I enhanced my patch because it was not generalized for instructions with N delay_slots. Mostly OK, though there are some formatting nits that need to be corrected. We have whitespace around arithmetic, logical and comparison operators

RE: try_merge_delay_insn with delay list > 1

2015-03-10 Thread BELBACHIR Selim
Me again :) I enhanced my patch because it was not generalized for instructions with N delay_slots. Selim try_merge_patch2 Description: try_merge_patch2

try_merge_delay_insn with delay list > 1

2015-03-10 Thread BELBACHIR Selim
Hi, I'm still working on a private backend on gcc 4.9.2. My processor provides instructions with 2 delay slots. I'm well aware that this feature is very uncommon and not fully tested. Nevertheless I submit the problem and the solution I've found. The bug is located in the function try_merge_de