Re: [PATCH] lto: Fix up lto location streaming

2020-09-10 Thread Richard Biener
On September 10, 2020 8:39:20 PM GMT+02:00, Jakub Jelinek wrote: >On Thu, Sep 10, 2020 at 05:32:58PM +0200, Jakub Jelinek via Gcc-patches >wrote: >> 2020-09-10 Jakub Jelinek >> >> * lto-streamer-out.c (collect_block_tree_leafs): Recurse on >> root rather than BLOCK_SUBBLOCKS (root).

Re: [PATCH] lto: Fix up lto location streaming

2020-09-10 Thread Jakub Jelinek via Gcc-patches
On Thu, Sep 10, 2020 at 05:32:58PM +0200, Jakub Jelinek via Gcc-patches wrote: > 2020-09-10 Jakub Jelinek > > * lto-streamer-out.c (collect_block_tree_leafs): Recurse on > root rather than BLOCK_SUBBLOCKS (root). > > --- gcc/lto-streamer-out.c.jj 2020-09-10 15:52:36.401413518 +0200

Re: [PATCH] lto: Fix up lto location streaming

2020-09-10 Thread Jakub Jelinek via Gcc-patches
On Thu, Sep 10, 2020 at 04:09:24PM +0200, Richard Biener wrote: > So we're usually streaming the bits and the tree portion of SCCs > separately so I really wonder how we can end up with recursion here? > For stmts they should only refer to BLOCKs also in the BLOCK tree > which is streamed in before

Re: [PATCH] lto: Fix up lto location streaming

2020-09-10 Thread Richard Biener
On Thu, 10 Sep 2020, Jakub Jelinek wrote: > Hi! > > When I've tried to backport recent LTO changes of mine, I've ran into > FAIL: g++.dg/ubsan/align-3.C -O2 -flto -fno-use-linker-plugin > -flto-partition=none output pattern test > FAIL: g++.dg/ubsan/align-3.C -O2 -flto -fuse-linker-plugin

[PATCH] lto: Fix up lto location streaming

2020-09-10 Thread Jakub Jelinek via Gcc-patches
Hi! When I've tried to backport recent LTO changes of mine, I've ran into FAIL: g++.dg/ubsan/align-3.C -O2 -flto -fno-use-linker-plugin -flto-partition=none output pattern test FAIL: g++.dg/ubsan/align-3.C -O2 -flto -fuse-linker-plugin -fno-fat-lto-objects output pattern test regressions t