Re: [PATCH] [SPARC] Disable muldf3_extend for LEON/LEON3

2017-07-25 Thread Sebastian Huber
On 25/07/17 15:42, Eric Botcazou wrote: What is your opinion with respect to a -mno-fsmuld option or something similar? Far better in my opinion (at least for LEON3). Attached is a variant with this option. It touches a lot more files. -- Sebastian Huber, embedded brains GmbH Address : Dor

Re: [PATCH] [SPARC] Disable muldf3_extend for LEON/LEON3

2017-07-25 Thread Sebastian Huber
On 25/07/17 16:12, Sebastian Huber wrote: On 25/07/17 15:42, Eric Botcazou wrote: What is your opinion with respect to a -mno-fsmuld option or something similar? Far better in my opinion (at least for LEON3). How should I add this option? For example: diff --git a/gcc/config/sparc/sparc.o

Re: [PATCH] [SPARC] Disable muldf3_extend for LEON/LEON3

2017-07-25 Thread Sebastian Huber
On 25/07/17 15:42, Eric Botcazou wrote: What is your opinion with respect to a -mno-fsmuld option or something similar? Far better in my opinion (at least for LEON3). How should I add this option? For example: diff --git a/gcc/config/sparc/sparc.opt b/gcc/config/sparc/sparc.opt index ae63d2

Re: [PATCH] [SPARC] Disable muldf3_extend for LEON/LEON3

2017-07-25 Thread Eric Botcazou
> What is your opinion with respect to a -mno-fsmuld option or something > similar? Far better in my opinion (at least for LEON3). -- Eric Botcazou

Re: [PATCH] [SPARC] Disable muldf3_extend for LEON/LEON3

2017-07-25 Thread Eric Botcazou
> Yes, everyone using LEON/LEON3 will no longer benefit from the FSMULD > after this patch. This was discussed with Gaisler. We have to make a > trade-off between correctness, performance and the number of GCC options. IMO we shouldn't introduce regressions like this, i.e. people should never be

Re: [PATCH] [SPARC] Disable muldf3_extend for LEON/LEON3

2017-07-25 Thread Sebastian Huber
On 25/07/17 14:59, Sebastian Huber wrote: On 25/07/17 14:59, Eric Botcazou wrote: Prominent LEON2 chips like the AT697E and AT697F do not properly support the FSMULD instruction. LEON3 targets using the GRFPU-lite floating point unit do not support the FSMULD in hardware. Disable this instr

Re: [PATCH] [SPARC] Disable muldf3_extend for LEON/LEON3

2017-07-25 Thread Sebastian Huber
On 25/07/17 14:59, Eric Botcazou wrote: Prominent LEON2 chips like the AT697E and AT697F do not properly support the FSMULD instruction. LEON3 targets using the GRFPU-lite floating point unit do not support the FSMULD in hardware. Disable this instruction for for all LEON/LEON3 targets for sim

Re: [PATCH] [SPARC] Disable muldf3_extend for LEON/LEON3

2017-07-25 Thread Eric Botcazou
> Prominent LEON2 chips like the AT697E and AT697F do not properly support > the FSMULD instruction. LEON3 targets using the GRFPU-lite floating > point unit do not support the FSMULD in hardware. Disable this > instruction for for all LEON/LEON3 targets for simplicity. And penalize everyone els

[PATCH] [SPARC] Disable muldf3_extend for LEON/LEON3

2017-07-25 Thread Sebastian Huber
Prominent LEON2 chips like the AT697E and AT697F do not properly support the FSMULD instruction. LEON3 targets using the GRFPU-lite floating point unit do not support the FSMULD in hardware. Disable this instruction for for all LEON/LEON3 targets for simplicity. This leads to a code size increas