Re: [PATCH][AArch64] Restrict POST_INC operand in aarch64_simd_mem_operand_p to register

2017-11-20 Thread Dominik Inführ
Thanks for the review! Could you also please commit this patch for me? I don’t have commit rights. Thanks, Dominik > On 17 Nov 2017, at 23:18, James Greenhalgh wrote: > > On Tue, Oct 31, 2017 at 02:47:29PM +0100, Dominik Inführ wrote: >> Hi, >> >> I have a custom optimization pass, that moves

Re: [PATCH][AArch64] Restrict POST_INC operand in aarch64_simd_mem_operand_p to register

2017-11-17 Thread James Greenhalgh
On Tue, Oct 31, 2017 at 02:47:29PM +0100, Dominik Inführ wrote: > Hi, > > I have a custom optimization pass, that moves an expression into an > POST_INC-expression. GCC then ICE’s in df-scan.c since it expects REG_P to be > true for POST_INC’s operand. aarch64_simd_mem_operand_p doesn’t seem to ch

Re: [PATCH][AArch64] Restrict POST_INC operand in aarch64_simd_mem_operand_p to register

2017-11-10 Thread Dominik Inführ
Ping > On 31 Oct 2017, at 14:47, Dominik Inführ > wrote: > > Hi, > > I have a custom optimization pass, that moves an expression into an > POST_INC-expression. GCC then ICE’s in df-scan.c since it expects REG_P to be > true for POST_INC’s operand. aarch64_simd_mem_operand_p doesn’t seem to c

Re: [PATCH][AArch64] Restrict POST_INC operand in aarch64_simd_mem_operand_p to register

2017-10-31 Thread Dominik Inführ
> On 31 Oct 2017, at 15:47, Kyrill Tkachov wrote: > > > On 31/10/17 14:44, Dominik Inführ wrote: >>> On 31 Oct 2017, at 15:10, Kyrill Tkachov >>> wrote: >>> >>> [cc'ing aarch64 maintainers] >>> >>> Hi Dominik, >>> >>> On 31/10/17 13:47, Dominik Inführ wrote: Hi, I have a cu

Re: [PATCH][AArch64] Restrict POST_INC operand in aarch64_simd_mem_operand_p to register

2017-10-31 Thread Kyrill Tkachov
On 31/10/17 14:44, Dominik Inführ wrote: On 31 Oct 2017, at 15:10, Kyrill Tkachov wrote: [cc'ing aarch64 maintainers] Hi Dominik, On 31/10/17 13:47, Dominik Inführ wrote: Hi, I have a custom optimization pass, that moves an expression into an POST_INC-expression. GCC then ICE’s in df-scan

Re: [PATCH][AArch64] Restrict POST_INC operand in aarch64_simd_mem_operand_p to register

2017-10-31 Thread Dominik Inführ
> On 31 Oct 2017, at 15:10, Kyrill Tkachov wrote: > > [cc'ing aarch64 maintainers] > > Hi Dominik, > > On 31/10/17 13:47, Dominik Inführ wrote: >> Hi, >> >> I have a custom optimization pass, that moves an expression into an >> POST_INC-expression. GCC then ICE’s in df-scan.c since it expect

Re: [PATCH][AArch64] Restrict POST_INC operand in aarch64_simd_mem_operand_p to register

2017-10-31 Thread Kyrill Tkachov
[cc'ing aarch64 maintainers] Hi Dominik, On 31/10/17 13:47, Dominik Inführ wrote: Hi, I have a custom optimization pass, that moves an expression into an POST_INC-expression. GCC then ICE’s in df-scan.c since it expects REG_P to be true for POST_INC’s operand. aarch64_simd_mem_operand_p does

[PATCH][AArch64] Restrict POST_INC operand in aarch64_simd_mem_operand_p to register

2017-10-31 Thread Dominik Inführ
Hi, I have a custom optimization pass, that moves an expression into an POST_INC-expression. GCC then ICE’s in df-scan.c since it expects REG_P to be true for POST_INC’s operand. aarch64_simd_mem_operand_p doesn’t seem to check POST_INC’s operand. Here is a patch that fixes this for me, althoug