[PATCH] Remove struct map from plugin-nvptx

2016-07-12 Thread Cesar Philippidis
I'm not sure why the nvptx offloading plugin introduced struct map in the first place, but its current usage is both unnecessary and buggy. For instance, it doesn't properly free data mappings, and that results in bogus duplicate data mapping errors. This patch removes that struct altogether. Jim

Re: [PATCH] - improve sprintf buffer overflow detection (middle-end/49905)

2016-07-12 Thread Martin Sebor
On 07/12/2016 10:12 AM, Manuel López-Ibáñez wrote: On 12/07/16 16:59, Martin Sebor wrote: You're probably right. I suspect I have a tendency to overuse the quotes (e.g, the -Wplacement-new warning also quotes the sizes). If there aren't yet (I vague recall coming across something on the GCC Wi

Re: [PATCH, rs6000] Fix PR target/71733, ICE with -mcpu=power9 -mno-vsx

2016-07-12 Thread Alan Modra
On Tue, Jul 12, 2016 at 04:17:23PM +0200, Ulrich Weigand wrote: > Alan Modra wrote: > > The problem now is that this passes rs6000_legitimate_address_p due to > > mode_supports_vsx_dform_quad and quad_address_p being true. That > > doesn't seem correct for -mno-vsx. > > Hmm, I see in rs6000_optio

Re: [patch, fortran] PR66310 Problems with intrinsic repeat for large number of copies

2016-07-12 Thread William Clodius
A minor question. Why with Fortran’s counted strings are you appending a null character? > On Jul 11, 2016, at 6:54 PM, Jerry DeLisle wrote: > > Attached patch sets a limit of huge-1 on the character count to avoid the > integer wrap. > > Regression tested with adjustment to test case. > > OK

Re: [PING^3] Re: Some fixes for autofdo test cases

2016-07-12 Thread Andi Kleen
Andi Kleen writes: Ping^3! > Andi Kleen writes: > > Ping^2! > >> Andi Kleen writes: >> >> Ping! >> >>> This fixes some of the problems with profile test cases running with autofdo >>> There are still remaining failures that need to be addressed, but this is >>> the >>> low hanging fruit. >>>

Re: [PATCH, rs6000] Fix PR target/71733, ICE with -mcpu=power9 -mno-vsx

2016-07-12 Thread Alan Modra
This is what I've bootstrapped and regression tested on powerpc64le-linux. I'm using Peter's testcases from this thread rather than the one in the original patch submission, because that one relies on -O0 not reducing the function down to a nop. OK to apply? PR target/71733 gcc/

<    1   2