On 15/01/14 10:45, Terry Guo wrote: > > >> -----Original Message----- >> From: Richard Earnshaw >> Sent: Wednesday, January 15, 2014 5:54 PM >> To: Terry Guo >> Cc: gcc-patches@gcc.gnu.org >> Subject: Re: [GCC, ARM] Backport trunk fix to 4.8 branch to properly > handle >> rtx of ARM PLD instruction >> >> On 15/01/14 09:23, Terry Guo wrote: >>> Hi there, >>> >>> With trunk enhancement at >>> http://gcc.gnu.org/ml/gcc-patches/2013-11/msg00533.html, gcc can >>> properly handle PLD rtx. Otherwise the PLD rtx will be treated as SET >>> rtx and gcc will end up with ICE. The attached patch intends to back >>> port this enhancement to 4.8 branch. Tested with gcc regression test, >>> no new regressions. Is it ok to back port? >>> >>> BR, >>> Terry >>> >>> 2014-01-15 Terry Guo <terry....@arm.com> >>> >>> Backported from mainline r204575 and applied to file arm.c. >>> 2013-11-08 James Greenhalgh <james.greenha...@arm.com> >>> >>> * config/arm/aarch-common.c >>> (search_term): New typedef. >>> (shift_rtx_costs): New array. >>> (arm_rtx_shift_left_p): New. >>> (arm_find_sub_rtx_with_search_term): Likewise. >>> (arm_find_sub_rtx_with_code): Likewise. >>> (arm_early_load_addr_dep): Add sanity checking. >>> (arm_no_early_alu_shift_dep): Likewise. >>> (arm_no_early_alu_shift_value_dep): Likewise. >>> (arm_no_early_mul_dep): Likewise. >>> (arm_no_early_store_addr_dep): Likewise. >>> >> >> Is there a PR for this? >> > > No. It is firstly found on arm/embedded-4_8-branch and then found on > upstream 4.8 branch. The trunk hasn't such issue. Do I need to report a PR > against 4.8? If so, I am willing to do it along with the test case. > > BR, > Terry >
Preferably, particularly since you haven't supplied a testcase. R.
