Re: [PATCH][1/3] Re-submission of Altera Nios II port, gcc parts

2013-12-27 Thread Chung-Lin Tang
On 13/12/23 12:54 AM, Chung-Lin Tang wrote: >> Other than these two, I think this can go in. >> > Bernd > Attached is the updated patch for the compiler. > > Since Bernd is a Global Reviewer, am I clear for committing the port > now? (including the testsuite and libgcc parts) I will be taking Ber

[Patch, trivial] Fix a potential bug of cost_classes_eq in ira-costs.c

2013-12-27 Thread Yangfei (Felix)
Hi Vladimir, I am trying to fix a potential bug of cost_classes_eq in ira-costs.c. This patch is for gcc-4.8 branch and should work for gcc-4.9. Library function memcmp return 0 if no difference for the two compared data. Please take a look. /* Compares cost classes info V1 and V2. */

[Patch, trivial] PR 56653: Fix warning when verifying checksums from MD5SUMS file in tarballs

2013-12-27 Thread Dmitry Gorbachev
This patch is to fix `md5sum: WARNING: 1 line is improperly formatted' thing. 2013-12-28 Dmitry Gorbachev PR 56653 * gcc_release: Add an extra `#' character to the comment header of MD5SUMS. *** maintainer-scripts/gcc_release --- maintainer-scripts/gcc_release **

libgo patch committed: Use DialTimeout in self-connect test

2013-12-27 Thread Ian Lance Taylor
This patch from Uros Bizjak changes TestSelfConnect to use DialTimeout, to avoid a problem where some operating systems take a long time to fail a connection to localhost. This patch adds a timeout to make the connection fail quickly. This is PR 59506. Ran Go testsuite on x86_64-unknown-linux-gn

Rb tree node recycling patch

2013-12-27 Thread François Dumont
Hi Here is a patch to add recycling of Rb tree nodes when possible. I replaced the _Rb_tree::_M_move_assign with a move assignment operator to benefit from: - move of elements when the whole data structure cannot be moved - faster data structure cloning rather than full regeneration of

std::vector move assign patch

2013-12-27 Thread François Dumont
Hi Here is a patch to fix an issue in normal mode during the move assignment. The destination vector allocator instance is moved too during the assignment which is wrong. As I discover this problem while working on issues with management of safe iterators during move operations this

Re: [PATCH i386 6/8] [AVX-512] Add builtins/intrinsics.

2013-12-27 Thread Uros Bizjak
On Fri, Dec 27, 2013 at 4:04 PM, Kirill Yukhin wrote: > On 18 Dec 18:16, Uros Bizjak wrote: >> the patch is OK (with above mentioned changes) for mainline. > Thanks, > > One more nit. It seems that currently vectorizer expects mask_type to > be equal to operand_type, which doesn't hold for AVX-51

Re: [PATCH i386 6/8] [AVX-512] Add builtins/intrinsics.

2013-12-27 Thread Kirill Yukhin
Hello, On 18 Dec 18:16, Uros Bizjak wrote: > the patch is OK (with above mentioned changes) for mainline. Thanks, One more nit. It seems that currently vectorizer expects mask_type to be equal to operand_type, which doesn't hold for AVX-512F. So, I'll comment out like this: @@ -34677,6 +36423,3

Re: [PATCH] Fix for PR59585

2013-12-27 Thread Yury Gribov
Jakub wrote: > Ok, I guess I can live with the original patch then. r206219. -Y

Re: [C++ Patch] Fix __is_base_of vs incomplete types

2013-12-27 Thread Paolo Carlini
Hi, On 12/12/2013 03:19 PM, Jason Merrill wrote: I wouldn't expect it to cause problems. Ok. Then I looked a bit into this and something is making me nervous: lookup_base wants to return a binfo, not a type. Thus what do we do when the type is incomplete and there is no binfo? Something like t

Re: RFA: fix libstdc++ regression, simulator timeout from r205810

2013-12-27 Thread Yufeng Zhang
Many thanks for your effort in fixing the issue. I can confirm that the new tests pass on arm-eabi using qemu as the simulator. Thanks, Yufeng P.s. Wish you have nice holiday break and happy new year! On 12/20/13 01:12, Hans-Peter Nilsson wrote: Here's a patch that splits up 20_util/hash/chi

Update GCC 4.9 changes.html

2013-12-27 Thread H.J. Lu
Hi, This patch adds Intel microarchitecture changes to GCC 4.9 changes.html. OK to install? Thanks. -- H.J. -- Index: changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.9/changes.html,v retrieving revision 1.49 diff -u -p -r1

Re: RFC Asan instrumentation control

2013-12-27 Thread Jakub Jelinek
Hi! On Fri, Dec 27, 2013 at 03:06:06PM +0400, Yury Gribov wrote: > We did exactly this for majority of tests except for Ah, sorry, I saw one test and didn't check carefully the rest of them. > * c-c++-common/asan/use-after-return-1.c (back-ported from Clang, no UB) Well, it is still UB, but one

Re: RFC Asan instrumentation control

2013-12-27 Thread Yury Gribov
Jakub wrote: IMNSHO it is far better to just change those tests into compile time tests and verify assembly using dg-final scan-assembler. We did exactly this for majority of tests except for * c-c++-common/asan/use-after-return-1.c (back-ported from Clang, no UB) * c-c++-common/asan/no-asan-st

Re: [PATCH] Fix for PR59585

2013-12-27 Thread Jakub Jelinek
On Tue, Dec 24, 2013 at 10:05:37AM +0400, Yury Gribov wrote: > Yury wrote: > >> Still sounds like a bug elsewhere to me. > > > > Let me investigate this deeper tomorrow (rebuilding fresh Dg, etc.). > > So I've double-checked that this is a problem with trunk DejaGNU > rsh.exp script removing trail

Re: RFC Asan instrumentation control

2013-12-27 Thread Jakub Jelinek
On Tue, Dec 24, 2013 at 02:10:57PM +0400, Maxim Ostapenko wrote: > >As for the tests, I'm afraid I don't like them at all. > >If anything, it ought to be dg-do compile tests where you say > >scan assembly or some dump, but having runtime testcases that > >trigger undefined behavior that isn't detec

Re: [PATCH, committed] Fix PR 57422

2013-12-27 Thread Jakub Jelinek
On Fri, Dec 27, 2013 at 02:11:13PM +0400, Andrey Belevantsev wrote: > >Testcase is very small. Why not add it? > > Frankly, I think that the chances of this test uncovering similar > issues in the future are very small. It needs lots of options to > make it trigger and even with this a specific r

Re: [PATCH, committed] Fix PR 57422

2013-12-27 Thread Andrey Belevantsev
On 23.12.2013 16:24, H.J. Lu wrote: On Sun, Dec 22, 2013 at 10:52 PM, Andrey Belevantsev wrote: Hello, As described in the PR, the ICE reason was the typo made when introducing calls to add_hard_reg_set. Fixed by the first attached patch, bootstrapped and tested on both ia64 and x86_64, commi