This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] combine: distribute_notes again (PR69567, PR64682)
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Segher Boessenkool <segher at kernel dot crashing dot org>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Sat, 6 Feb 2016 09:54:25 -0800
- Subject: Re: [PATCH] combine: distribute_notes again (PR69567, PR64682)
- Authentication-results: sourceware.org; auth=none
- References: <9f15493d0cbf1a52a475998ffa22f1fc26f5e275 dot 1454366721 dot git dot segher at kernel dot crashing dot org> <20160204232830 dot GA9829 at gate dot crashing dot org>
On Thu, Feb 4, 2016 at 3:28 PM, Segher Boessenkool
<segher@kernel.crashing.org> wrote:
> On Tue, Feb 02, 2016 at 11:32:08AM +0000, Segher Boessenkool wrote:
>> As it happens the patch I did over a year ago for PR64682 isn't quite
>> correct. This is PR69567. This fixes it.
>>
>> Tested on the separate testcases; also did bootstrap + testsuite on
>> powerpc64-linux and x86_64-linux. I'll commit this tomorrow or so if
>> no one sees something wrong with it.
>>
>>
>> 2016-02-02 Segher Boessenkool <segher@kernel.crashing.org>
>>
>> PR rtl-optimization/64682
>> PR rtl-optimization/69567
>> * combine.c (distribute_notes) <REG_DEAD>: Place the death note
>> before I2 only if the register is both used and set in I2.
>
> Committed to mainline and 5 now.
>
This caused:
FAIL: gcc.c-torture/execute/pr64682.c -O2 execution test
FAIL: gcc.c-torture/execute/pr64682.c -O2 -flto
-fno-use-linker-plugin -flto-partition=none execution test
FAIL: gcc.c-torture/execute/pr64682.c -O2 -flto -fuse-linker-plugin
-fno-fat-lto-objects execution test
FAIL: gcc.c-torture/execute/pr64682.c -O3 -fomit-frame-pointer execution test
FAIL: gcc.c-torture/execute/pr64682.c -O3 -fomit-frame-pointer
-funroll-all-loops -finline-functions execution test
FAIL: gcc.c-torture/execute/pr64682.c -O3 -fomit-frame-pointer
-funroll-loops execution test
FAIL: gcc.c-torture/execute/pr64682.c -O3 -g execution test
on x86 on GCC 5 branch. Trunk is OK. Did you miss a backport from
trunk?
--
H.J.