This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, PR target/67761] Fix i686-*-* bootstrap comparison failure
- From: Ilya Enkovich <enkovich dot gnu at gmail dot com>
- To: Uros Bizjak <ubizjak at gmail dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 30 Sep 2015 16:10:52 +0300
- Subject: Re: [PATCH, PR target/67761] Fix i686-*-* bootstrap comparison failure
- Authentication-results: sourceware.org; auth=none
- References: <CAFULd4b=dC+YLcdet0qNUG_gtVvQ5XdYQrkifmo91F7KPZHLBw at mail dot gmail dot com>
2015-09-30 9:06 GMT+03:00 Uros Bizjak <ubizjak@gmail.com>:
> Hello!
>
>> My recenttly introduced STV pass doesn't skip debug instructions and it causes transformation
>> (mistly cost computation) depending on debug info. It causes bootstrap comparison failure. This
>> patch fixes. Bootstrapped for i686-linux. Testing for x86_64-unknown-linux-gnu{,m32} is in
>> progress. OK for trunk if pass?
>
> IMO, it would be also beneficial to bootstrap with slm default
> architecture, so new code paths get some coverage via bootstrap.
I bootstrapped with --with-cpu=slm also.
>
>> gcc/
>>
>> 2015-09-29 Ilya Enkovich <enkovich.gnu@gmail.com>
>>
>> * config/i386/i386.c (scalar_chain::analyze_register_chain): Ignore
>> debug insns.
>> (scalar_chain::convert_reg): Likewise.
>>
>> gcc/testsuite/
>>
>> 2015-09-29 Ilya Enkovich <enkovich.gnu@gmail.com>
>>
>> * gcc.target/i386/pr67761.c: New test.
>
> OK.
Thanks!
Ilya
>
> Thanks,
> Uros.