Re: [PATCH][AArch64] PR target/70809: Delete aarch64_vmls pattern

2016-05-20 Thread Kyrill Tkachov
On 17/05/16 12:04, James Greenhalgh wrote: On Tue, May 17, 2016 at 11:37:57AM +0100, Kyrill Tkachov wrote: Hi all, The aarch64_vmls pattern claims to perform a normal vector floating-point multiply-subtract but in fact performs a fused multiply-subtract. This is fine when -ffp-contract=fast, b

Re: [PATCH][AArch64] PR target/70809: Delete aarch64_vmls pattern

2016-05-17 Thread James Greenhalgh
On Tue, May 17, 2016 at 11:37:57AM +0100, Kyrill Tkachov wrote: > Hi all, > > The aarch64_vmls pattern claims to perform a normal vector > floating-point multiply-subtract but in fact performs a fused > multiply-subtract. This is fine when -ffp-contract=fast, but it's not guarded > on anything so

[PATCH][AArch64] PR target/70809: Delete aarch64_vmls pattern

2016-05-17 Thread Kyrill Tkachov
Hi all, The aarch64_vmls pattern claims to perform a normal vector floating-point multiply-subtract but in fact performs a fused multiply-subtract. This is fine when -ffp-contract=fast, but it's not guarded on anything so will generate the FMLS instruction even when -ffp-contract=off. The solu