Re: [PATCH] look harder for MEM_REF operand equality to avoid -Wstringop-truncation (PR 84561)

2018-09-15 Thread Bernd Edlinger
On 9/14/18, Martin Sebor wrote: > As I said above, this happens during the dom walk in the ccp > pass: > > substitute_and_fold_dom_walker walker (CDI_DOMINATORS, this); > walker.walk (ENTRY_BLOCK_PTR_FOR_FN (cfun)); > > The warning is issued during the walker.walk() call as > strncpy is being

Re: [PATCH] Make strlen range computations more conservative

2018-09-15 Thread Bernd Edlinger
Hi, this is an update on my strlen range patch (V7). Again re-based and retested to current trunk. I am aware that Martin wants to re-factor the interface of get_range_strlen and have no objections against, but I'd suggest that to be a follow-up patch. I might suggest to rename one of the two g

Fix PR middle-end/86864

2018-09-15 Thread Eric Botcazou
pared for a BARRIER before and after a JUMP_TABLE_DATA. 2018-09-15 Eric Botcazou * gcc.c-torture/compile/20180915-1.c: New test. -- Eric BotcazouIndex: cfgexpand.c === --- cfgexpand.c (revision 264202) +++ cfgexp

vector _M_start and 0 offset

2018-09-15 Thread Marc Glisse
Hello, as explained in the PR, the implementation of vector is weirdly wasteful. Preserving the ABI prevents from changing much for now, but this small tweak can help the compiler remove quite a bit of dead code. I think most other direct uses of _M_start are in constructors where the offset

bootstrap -O3 failure libgo on x64_86

2018-09-15 Thread graham stott via gcc-patches
Hi Fails due to undefind symbols runtime.* O2 works

Re: bootstrap -O3 failure libgo on x64_86

2018-09-15 Thread Ian Lance Taylor via gcc-patches
On Sat, Sep 15, 2018 at 7:12 AM, graham stott via gcc-patches wrote: > Hi > Fails due to undefind symbols runtime.* > O2 works I tried my best guess at recreating the problem, and everything worked fine. Please tell us exactly what you did and exactly what happened. Thanks. Ian

[PATCH, rs6000, committed] Use correct ABI type in mmintrin.h and xmmintrin.h

2018-09-15 Thread Bill Schmidt
Hi, Jinsong Ji reported that these header files are using the non-ABI type __int128_t rather than __int128. This patch from Jinsong corrects this. Bootstrapped and tested on powerpc64le-linux-gnu with no regressions, committed as obvious. Thanks, Bill 2018-09-15 Jinsong Ji Bill

Re: [PATCH, rs6000, committed] Use correct ABI type in mmintrin.h and xmmintrin.h

2018-09-15 Thread Segher Boessenkool
On Sat, Sep 15, 2018 at 03:28:17PM -0500, Bill Schmidt wrote: > Jinsong Ji reported that these header files are using the non-ABI type > __int128_t rather than __int128. This patch from Jinsong corrects this. > Bootstrapped and tested on powerpc64le-linux-gnu with no regressions, > committed as ob

[wwwdocs] gcc-3.1/gcj-status.html -- move to full HTML

2018-09-15 Thread Gerald Pfeifer
Similar to what I did with gcc-3.3/gcj-status.html a few days ago. Committed. Gerald Index: gcc-3.1/gcj-status.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-3.1/gcj-status.html,v retrieving revision 1.6 diff -u -r1.6 gcj-status.htm

[wwwdocs] remove last use of cellspacing="0"

2018-09-15 Thread Gerald Pfeifer
We had this left on the main page. I makes a slight visual difference, but not signficant, and I plan on replacing the formatting via a table mid term. Applied. Gerald Index: style.mhtml === RCS file: /cvs/gcc/wwwdocs/htdocs/style.

[doc] doc/service.texi: Switch the fsf.org link to https.

2018-09-15 Thread Gerald Pfeifer
Committed. Gerald 2018-09-16 Gerald Pfeifer * doc/service.texi (Service): Switch the fsf.org link to https. Index: doc/service.texi === --- doc/service.texi(revision 264342) +++ doc/service.texi(working copy) @@

Re: [PATCH] PR86957

2018-09-15 Thread Indu Bhagat
Thanks for the reviews. I have incorporated all but one (See below; its the change in the warning's brief summary in common.opt) in the patch. In this patch, 1. -Wmissing-profile is a warning by default and is ON by default with -fprofile-use 2. Attached pr86957-missing-profile-diagnostic-2