This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, ARM, v2] Fix PR target/59142: internal compiler error while compiling OpenCV 2.4.7
- From: Charles Baylis <charles dot baylis at linaro dot org>
- To: Richard Earnshaw <rearnsha at arm dot com>, Christophe Lyon <christophe dot lyon at linaro dot org>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Ramana Radhakrishnan <Ramana dot Radhakrishnan at arm dot com>
- Date: Thu, 16 Jan 2014 18:40:22 +0000
- Subject: Re: [PATCH, ARM, v2] Fix PR target/59142: internal compiler error while compiling OpenCV 2.4.7
- Authentication-results: sourceware.org; auth=none
- References: <CADnVucA91aS83S46Ep+4fW+bYr6OTnVDosK7YAq-LntMt3O8FQ at mail dot gmail dot com> <529364A1 dot 8040403 at arm dot com> <CADnVucAAp8wde5sGy1tX++E27WMG5QDy0L7U3=A3uye3Yyazhg at mail dot gmail dot com> <5294772E dot 70309 at arm dot com> <20131219153809 dot GA18927 at sale> <52B31B28 dot 1020309 at arm dot com> <CADnVucDotL_Z8gau85aQ1_P081uhw31=j3-+6A9Z4UhwpOT+iw at mail dot gmail dot com> <52B4458B dot 1070203 at arm dot com>
On 20 December 2013 13:26, Richard Earnshaw <rearnsha@arm.com> wrote:
> On 19/12/13 17:40, Charles Baylis wrote:
>> Is it ok for 4.8, and should it be considered for 4.7?
>>
>
> Yes, provided it passes testing on those releases.
Results of testing 4.8:
All 3 patches:
0001-PR-target-59142-vfp_hard_register_operand.patch
0002-PR-target-59142-arm_hard_general_register_operand.patch
0003-PR-target-59142-low_register_operand.patch
apply correctly, and I have verified that ldmstm.md is correctly
patched and does not need to be regenerated and have tested that the
compiler bootstraps and passes make check in a arm-linux-gnueabihf
configuration on a chromebook.
Results of testing 4.7:
Only the following 2 patches should be applied as patch 0001 modifies
a pattern which does not exist on the 4.7 branch.
0002-PR-target-59142-arm_hard_general_register_operand.patch
0003-PR-target-59142-low_register_operand.patch
I have verified that ldmstm.md is correctly patched and does not need
to be regenerated and have tested that the compiler bootstraps in a
arm-linux-gnueabi configuration on a chromebook.
I think this is OK to be committed to both branches?