Re: [PATCH][AArch64] Fix ICE caused in aarch64_simd_valid_immediate

2017-10-06 Thread Tamar Christina
; Marcus Shawcroft; James Greenhalgh Subject: Re: [PATCH][AArch64] Fix ICE caused in aarch64_simd_valid_immediate Hi Richard *** gcc/ChangeLog *** 2017-10-05 Sudakshina Das PR target/82440 * config/aarch64/predicates.md (aarch64_reg_or_orr_imm): Only call

Re: [PATCH][AArch64] Fix ICE caused in aarch64_simd_valid_immediate

2017-10-06 Thread Sudi Das
Das; Jakub Jelinek Cc: gcc-patches@gcc.gnu.org; nd; sell...@cavium.com; Marcus Shawcroft; James Greenhalgh Subject: Re: [PATCH][AArch64] Fix ICE caused in aarch64_simd_valid_immediate   On 06/10/17 12:01, Sudi Das wrote: > > Hi Jakub > > I have modified the entries: > >

Re: [PATCH][AArch64] Fix ICE caused in aarch64_simd_valid_immediate

2017-10-06 Thread Richard Earnshaw (lists)
1 AM > To: Sudi Das > Cc: gcc-patches@gcc.gnu.org; nd; sell...@cavium.com; Marcus Shawcroft; > Richard Earnshaw; James Greenhalgh > Subject: Re: [PATCH][AArch64] Fix ICE caused in aarch64_simd_valid_immediate > > On Fri, Oct 06, 2017 at 09:52:35AM +, Sudi Das wrote: >

Re: [PATCH][AArch64] Fix ICE caused in aarch64_simd_valid_immediate

2017-10-06 Thread Sudi Das
; sell...@cavium.com; Marcus Shawcroft; Richard Earnshaw; James Greenhalgh Subject: Re: [PATCH][AArch64] Fix ICE caused in aarch64_simd_valid_immediate   On Fri, Oct 06, 2017 at 09:52:35AM +, Sudi Das wrote: > This patch is a fix for PR 82440. > The predicates aarch64_reg_or_bic_i

Re: [PATCH][AArch64] Fix ICE caused in aarch64_simd_valid_immediate

2017-10-06 Thread Jakub Jelinek
On Fri, Oct 06, 2017 at 09:52:35AM +, Sudi Das wrote: > This patch is a fix for PR 82440. > The predicates aarch64_reg_or_bic_imm and aarch64_reg_or_orr_imm missed out on > checking for a CONST_VECTOR before calling aarch64_simd_valid_immediate > function. > Also I think James forgot to add th

[PATCH][AArch64] Fix ICE caused in aarch64_simd_valid_immediate

2017-10-06 Thread Sudi Das
Hi This patch is a fix for PR 82440. The predicates aarch64_reg_or_bic_imm and aarch64_reg_or_orr_imm missed out on checking for a CONST_VECTOR before calling aarch64_simd_valid_immediate function. Also I think James forgot to add the test cases in the original patch submitted. Testing done :