Re: [PATCH, rs6000] Add non-relative jump table support on Power Linux

2020-11-22 Thread HAO CHEN GUI via Gcc-patches
Hi, Gentle ping this: https://gcc.gnu.org/pipermail/gcc-patches/2020-October/556236.html Thanks. Gui Haochen On 6/11/2020 上午 9:02, HAO CHEN GUI wrote: Hi, Gentle ping this: https://gcc.gnu.org/pipermail/gcc-patches/2020-October/556236.html Thanks. Gui Haochen On 15/10/2020 下午 4:46,

[PATCH] [tree-optimization] Optimize or+and+or pattern to and+or

2020-11-22 Thread Eugene Rozenfeld via Gcc-patches
Simplify ((b | c) & a) | b to (a & c) | b. This fixes PR96679. Tested on x86_64-pc-linux-gnu. int f(int a, int b, int c) { return ((b | c) & a) | b; } Code without the patch: or edx,esi andedx,edi moveax,edx or eax,esi ret Code with the patch: andedi,edx mov

Re: Document --with-build-config=bootstrap-asan option.

2020-11-22 Thread Gerald Pfeifer
On Fri, 20 Nov 2020, Matthew Malcomson wrote: >>> The patch is okay modulo the question above. > Apologies for bringing up something from this far back, but I've just > noticed I never actually committed this patch. Actually thank you for doing so! (And I am known to sometimes bring up things

Re: [PATCH] libstdc++: Add C++2a synchronization support

2020-11-22 Thread Stephan Bergmann via Gcc-patches
On 20/11/2020 23:44, Thomas Rodgers wrote: Tested x86_64-pc-linux-gnu, committed. ...and there are multiple definition complaints from the linker because of two missing "include": index 7b2682a577e..23ab2018ca8 100644 --- a/libstdc++-v3/include/bits/atomic_wait.h +++

[PATCH] i386: Use SWI48DWI mode iterator for abs and maxmin [PR97873]

2020-11-22 Thread Uros Bizjak via Gcc-patches
Generate special double mode sequence also for TImode on 64bit targets. 2020-11-22 Uroš Bizjak PR target/97873 gcc/ * config/i386/i386.md (abs2): Use SWI48DWI mode iterator. (*abs2_doubleword): Use DWIH mode iterator. (3): Use SWI48DWI mode iterator. (*3_doubleword): Use

Re: [PATCH] libstdc++: Add C++2a synchronization support

2020-11-22 Thread Stephan Bergmann via Gcc-patches
On 20/11/2020 23:44, Thomas Rodgers wrote: Tested x86_64-pc-linux-gnu, committed. Clang complains: $ cat test.cc #include $ clang++ --gcc-toolchain=~/gcc/trunk/inst -std=c++20 -fsyntax-only test.cc In file included from test.cc:1: In file included from

[committed] d: Fix OutOfMemoryError thrown when appending to an array with a side effect (PR97889)

2020-11-22 Thread Iain Buclaw via Gcc-patches
Hi, When appending a character to an array, the result of that concat assignment was not the new value of the array, similarly, when appending an array to another array, side effects were evaluated in reverse to the expected order of evaluation. As of this change, the address of the left-hand

Re: Go patch committed: Change name mangling convention

2020-11-22 Thread Rainer Orth
Hi Ian, > This patch to the Go frontend and libgo changes the name mangling > convention. The previous convention (which was actually the second > one) turned out to be ambiguous when the path to a package contained a > dot; this is a common case, as many package paths are of the form >

[committed] Partially revert recent H8 patch for conditional branches

2020-11-22 Thread Jeff Law via Gcc-patches
    Partially revert recent H8 patch for conditional branches         So I'd forgotten an important tidbit on the H8 port.  Specifically     for a branch instruction, the target label must be operand 0 for     the length computations.         This really only affects the main conditional branch

Re: [PATCH v3 1/2] generate EH info for volatile asm statements (PR93981)

2020-11-22 Thread J.W. Jagersma via Gcc-patches
On 2020-11-21 12:27, J.W. Jagersma wrote: > ... > Another idea I had is to introduce a new operand modifier, eg. '-', which > would signify that the output *must* be considered clobbered on exception, > and it would be an error if a copy is not possible. Then the meaning of '+' > can be extended

[pushed] Darwin : Avoid a C++ ODR violation seen with LTO.

2020-11-22 Thread Iain Sandoe
Hi We have a similar code pattern in darwin-c.c to one in c-pragmas (most likely a cut & paste) with a struct type used locally to the TU. With C++ we need to rename the type to avoid an ODR violation. tested on x86_64-darwin19 ‘—with-build-config=bootstrap-lto-noplugin’ pushed to master,

Re: [PATCH] libstdc++: Add C++2a synchronization support

2020-11-22 Thread Jonathan Wakely via Gcc-patches
On Sun, 22 Nov 2020, 12:29 Iain Sandoe, wrote: > thanks for looking at this over the weekend. > > Jonathan Wakely via Gcc-patches wrote: > > > On Sat, 21 Nov 2020 at 23:55, David Edelsohn via Libstdc++ > > wrote: > >> I am seeing 93 new libstdc++ failures on AIX, even after Jonathan's > >>

Re: [PATCH] libstdc++: Add C++2a synchronization support

2020-11-22 Thread Iain Sandoe via Gcc-patches
thanks for looking at this over the weekend. Jonathan Wakely via Gcc-patches wrote: On Sat, 21 Nov 2020 at 23:55, David Edelsohn via Libstdc++ wrote: I am seeing 93 new libstdc++ failures on AIX, even after Jonathan's fixes. And a few c++ failures with similar symptoms. I'm not certain

Re: [Committed] patch fixing LRA crash on s390x

2020-11-22 Thread Jakub Jelinek via Gcc-patches
On Sun, Nov 15, 2020 at 12:08:22PM -0500, Vladimir Makarov via Gcc-patches wrote: > --- a/gcc/lra.c > +++ b/gcc/lra.c > @@ -1903,15 +1903,23 @@ lra_process_new_insns (rtx_insn *insn, rtx_insn > *before, rtx_insn *after, > { > /* We already made the edge no-critical in

[PATCH] Avoid atomic for guard acquire when that is expensive

2020-11-22 Thread Bernd Edlinger
Hi, this avoids the need to use -fno-threadsafe-statics on arm-none-eabi or working around that problem by supplying a dummy __sync_synchronize function which might just lead to silent code failure of the worst kind (non-reproducable, racy) at runtime, as was pointed out on previous discussions