Re: [C++ PATCH] Clear TREE_ADDRESSABLE on non-aggregate thunk arguments (PR c++/89187)

2019-02-05 Thread Jason Merrill
On 2/5/19 11:40 AM, Jakub Jelinek wrote: On Tue, Feb 05, 2019 at 11:16:04AM -0500, Jason Merrill wrote: --- gcc/cp/optimize.c.jj2019-01-21 23:32:43.0 +0100 +++ gcc/cp/optimize.c 2019-02-04 16:40:21.354179933 +0100 @@ -417,6 +417,12 @@ maybe_thunk_body (tree fn, bool force)

Re: [C++ PATCH] Clear TREE_ADDRESSABLE on non-aggregate thunk arguments (PR c++/89187)

2019-02-05 Thread Jakub Jelinek
On Tue, Feb 05, 2019 at 11:16:04AM -0500, Jason Merrill wrote: > > --- gcc/cp/optimize.c.jj2019-01-21 23:32:43.0 +0100 > > +++ gcc/cp/optimize.c 2019-02-04 16:40:21.354179933 +0100 > > @@ -417,6 +417,12 @@ maybe_thunk_body (tree fn, bool force) > > gcc_assert (clone_

Re: [C++ PATCH] Clear TREE_ADDRESSABLE on non-aggregate thunk arguments (PR c++/89187)

2019-02-05 Thread Jason Merrill
On 2/5/19 4:01 AM, Jakub Jelinek wrote: Hi! As the following testcase shows, for thunks the expansion code sometimes requires that arguments with gimple reg type aren't addressable, thunk needs to be simple passing of arguments directly to another call, not having extra statements in between tha

[C++ PATCH] Clear TREE_ADDRESSABLE on non-aggregate thunk arguments (PR c++/89187)

2019-02-05 Thread Jakub Jelinek
Hi! As the following testcase shows, for thunks the expansion code sometimes requires that arguments with gimple reg type aren't addressable, thunk needs to be simple passing of arguments directly to another call, not having extra statements in between that. In particular, for pass_by_reference a