Re: [C++ PATCH] Add tests for a const member and a reference member for launder.

2016-10-31 Thread Jason Merrill
OK, thanks. On Mon, Oct 31, 2016 at 12:13 PM, Ville Voutilainen wrote: > On 31 October 2016 at 15:31, Jason Merrill wrote: >> Good idea. You might put the reuse in a separate function in order to >> hide it from the optimizer. > > > Ok, new patch, tested on Linux-x64, ok for trunk? > > 2016-10-

Re: [C++ PATCH] Add tests for a const member and a reference member for launder.

2016-10-31 Thread Ville Voutilainen
On 31 October 2016 at 15:31, Jason Merrill wrote: > Good idea. You might put the reuse in a separate function in order to > hide it from the optimizer. Ok, new patch, tested on Linux-x64, ok for trunk? 2016-10-31 Ville Voutilainen Add tests for a const member and a reference member for

Re: [C++ PATCH] Add tests for a const member and a reference member for launder.

2016-10-31 Thread Jason Merrill
On Sun, Oct 30, 2016 at 2:53 PM, Ville Voutilainen wrote: > So, how about adding these? They should give us regression tests > that make sure launder does the right in in case some optimizations > attempt to reuse const/reference members. Good idea. You might put the reuse in a separate function

[C++ PATCH] Add tests for a const member and a reference member for launder.

2016-10-30 Thread Ville Voutilainen
So, how about adding these? They should give us regression tests that make sure launder does the right in in case some optimizations attempt to reuse const/reference members. Tested on Linux-x64. 2016-10-30 Ville Voutilainen Add tests for a const member and a reference member for launder.