On Fri, Dec 5, 2014 at 12:46 PM, Jeff Law <l...@redhat.com> wrote: > On 12/05/14 09:13, Martin Jambor wrote: >> >> Hi, >> >> at some point I lost an important division of bit offset by >> BITS_PER_UNIT in my alignment IPA-CP propagation patch. That lead to a >> few failures on i686 reported as PR 64192. >> >> This patch adds it together with a slight improvement of the guarding >> check which I suppose will never trigger but it does ensure the >> division will never loose information. >> >> I consider this change obvious and would really like to commit it >> before I leave for the weekend, so I will do so after it finishes >> bootstrapping and testing on i686. It has already passed bootstrap >> and testing on x86_64-linux. >> >> Thanks, >> >> Martin >> >> >> 2014-12-05 Martin Jambor <mjam...@suse.cz> >> >> PR ipa/64192 >> * ipa-prop.c (ipa_compute_jump_functions_for_edge): Convert >> alignment >> from bits to bytes after checking they are byte-aligned. > > OK. > > If you could add a testcase when you get back that'd be appreciated. >
It is covered by existing testcases. Those failed on Linux/i686: FAIL: gcc.c-torture/execute/pr37573.c -O3 -fomit-frame-pointer execution test FAIL: gcc.c-torture/execute/pr37573.c -O3 -fomit-frame-pointer -funroll-all-loops -finline-functions execution test FAIL: gcc.c-torture/execute/pr37573.c -O3 -fomit-frame-pointer -funroll-loops execution test FAIL: gcc.c-torture/execute/pr37573.c -O3 -g execution test FAIL: gcc.dg/vect/pr60196-1.c execution test FAIL: gcc.dg/vect/pr60196-1.c -flto -ffat-lto-objects execution test FAIL: gcc.dg/vect/vect-multitypes-11.c -flto -ffat-lto-objects execution test FAIL: gcc.dg/vect/vect-multitypes-12.c -flto -ffat-lto-objects execution test -- H.J.
