Re: RFC patch for #pragma ivdep

2013-10-08 Thread Jakub Jelinek
On Tue, Oct 08, 2013 at 08:51:50AM +0200, Tobias Burnus wrote: > --- a/gcc/cfgloop.c > +++ b/gcc/cfgloop.c > @@ -507,6 +507,39 @@ flow_loops_find (struct loops *loops) > loop->latch = latch; > } > } > + /* Search for ANNOTATE call with annot_expr_ivdep_kind; if foun

Cleanup patches

2013-10-08 Thread Thomas Schwinge
Hi! Here are a few cleanup patches, mostly in the realm of OpenMP, so Jakub gets a CC. OK to commit? libgomp/ * omp.h.in: Don't touch the user's namespace. --- libgomp/omp.h.in | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git libgomp/omp.h.in libgomp/omp.h.in in

Re: [PATCH][2 of 2] RTL expansion for zero sign extension elimination with VRP

2013-10-08 Thread Kugan
Ping~ Thanks, Kugan +2013-09-25 Kugan Vivekanandarajah + + * dojump.c (do_compare_and_jump): Generate rtl without + zero/sign extension if redundant. + * cfgexpand.c (expand_gimple_stmt_1): Likewise. + * gimple.c (gimple_assign_is_zero_sign_ext_redundant) : New +

[gomp4] Fix bootstrap

2013-10-08 Thread Jakub Jelinek
Hi! Got a warning turned into error during bootstrap that name might be used uninitialized. Fixed by not using it at all, there is no point to duplicate the clause names when we have omp_clause_code_name array. 2013-10-08 Jakub Jelinek * c-typeck.c (c_finish_omp_clauses): Remove name

[gomp4] Another affinity-1.c testcase fix

2013-10-08 Thread Jakub Jelinek
Hi! As reported by Vincenzo, the testcase didn't try to verify affinity behavior if _SC_NPROCESSORS_CONF returned value was smaller than kernel's internal mask size (e.g. because of CPU hotplug support). Fixed thusly: 2013-10-08 Jakub Jelinek * testsuite/libgomp.c/affinity-1.c (min_c

Re: [PATCH, PR 57748] Check for out of bounds access, Part 2

2013-10-08 Thread Eric Botcazou
> OK, what do you think of it now? My take on this is that the Proper Fix(tm) has been posted by Martin: http://gcc.gnu.org/ml/gcc-patches/2013-08/msg00082.html IMO it's a no-brainer, modulo the ABI concern. Everything else is more or less clever stuff to paper over this original compute_recor

[patch] Fix PR middle-end/58570

2013-10-08 Thread Eric Botcazou
Hi, this is a regression on the mainline introduced by my tree-ssa-alias.c change: 2013-04-17 Eric Botcazou * tree-ssa-alias.c (nonoverlapping_component_refs_of_decl_p): New. (decl_refs_may_alias_p): Add REF1 and REF2 parameters. Use nonoverlapping_component_refs_of_de

[ping] Fix PR bootstrap/58509

2013-10-08 Thread Eric Botcazou
http://gcc.gnu.org/ml/gcc-patches/2013-09/msg02000.html Thanks in advance. -- Eric Botcazou

Re: [PATCH] alternative hirate for builtin_expert

2013-10-08 Thread Ramana Radhakrishnan
>> Can someone comment / approve it quickly so that we get AArch32 and AArch64 >> linux cross-builds back up ? > > Ok. Applied for Dehao as r203269 . Tests on arm came back ok. Ramana > > Thanks, > Richard. > >> >> regards >> Ramana >> >>> >>> Honza >>> Dehao > > Honza >>>

RE: Fix scheduler ix86_issue_rate and ix86_adjust_cost for modern x86 chips

2013-10-08 Thread Gopalasubramanian, Ganesh
Hi Honza, I am planning to update the scheduler descriptions for bdver3 first. Attached is the patch. Please let me know your comments if any. Though I agree on merging bdver1/2 and bdver3 on most parts, the FP lines and decoding schemes are different. So, let me know how can I approach merging

[Patch] Fix the testcases that use bind_pic_locally

2013-10-08 Thread Vidya Praveen
Hello, There are several tests that use "dg-add-options bind_pic_locally" in order to add -fPIE or -fpie when -fPIC or -fpic are used respectively with the expecta- tion that -fPIE/-fpie will override -fPIC/-fpic. But this doesn't happen since since -fPIE/-fpie will be added before the -fPIC/-fpic

RE: [PATCH, PR 57748] Check for out of bounds access, Part 2

2013-10-08 Thread Bernd Edlinger
Hi, On Tue, 8 Oct 2013 10:01:37, Eric Botcazou wrote: > >> OK, what do you think of it now? > > My take on this is that the Proper Fix(tm) has been posted by Martin: > http://gcc.gnu.org/ml/gcc-patches/2013-08/msg00082.html > IMO it's a no-brainer, modulo the ABI concern. Everything else is more o

Re: [Patch] Fix the testcases that use bind_pic_locally

2013-10-08 Thread Jakub Jelinek
On Tue, Oct 08, 2013 at 10:14:59AM +0100, Vidya Praveen wrote: > There are several tests that use "dg-add-options bind_pic_locally" in order to > add -fPIE or -fpie when -fPIC or -fpic are used respectively with the expecta- > tion that -fPIE/-fpie will override -fPIC/-fpic. But this doesn't happen

Re: Fix scheduler ix86_issue_rate and ix86_adjust_cost for modern x86 chips

2013-10-08 Thread Jan Hubicka
> Hi Honza, > > I am planning to update the scheduler descriptions for bdver3 first. > Attached is the patch. Please let me know your comments if any. > > Though I agree on merging bdver1/2 and bdver3 on most parts, the FP lines and > decoding schemes are different. So, let me know how can I app

[2nd PING] [C++ PATCH] demangler fix (take 2)

2013-10-08 Thread Gary Benson
Hi all, This is a resubmission of my previous demangler fix [1] rewritten to avoid using hashtables and other libiberty features. From the above referenced email: d_print_comp maintains a certain amount of scope across calls (namely a stack of templates) which is used when evaluating references

Re: [patch] Add tree-ssa-loop.h and friends.

2013-10-08 Thread Richard Biener
On Mon, Oct 7, 2013 at 3:39 PM, Andrew MacLeod wrote: > On 10/07/2013 04:44 AM, Richard Biener wrote: >> >> On Thu, Oct 3, 2013 at 4:11 AM, Andrew MacLeod >> wrote: >>> >>> this patch consolidates tree-ssa-loop*.c files with new .h files as >>> required >>> (8 in total) >>> >>> A number of the pr

Re: Optimize callers using nonnull attribute

2013-10-08 Thread Richard Biener
On Mon, Oct 7, 2013 at 3:52 PM, Marc Glisse wrote: > On Mon, 7 Oct 2013, Richard Biener wrote: > >> On Mon, Oct 7, 2013 at 12:33 AM, Marc Glisse wrote: >>> >>> Hello, >>> >>> this patch asserts that when we call a function with the nonnull >>> attribute, >>> the corresponding argument is not zero

Re: [patch] Fix PR middle-end/58570

2013-10-08 Thread Richard Biener
On Tue, Oct 8, 2013 at 10:19 AM, Eric Botcazou wrote: > Hi, > > this is a regression on the mainline introduced by my tree-ssa-alias.c change: > > 2013-04-17 Eric Botcazou > > * tree-ssa-alias.c (nonoverlapping_component_refs_of_decl_p): New. > (decl_refs_may_alias_p): Add REF1

Re: [patch] Fix PR bootstrap/58509

2013-10-08 Thread Richard Biener
On Fri, Sep 27, 2013 at 1:17 PM, Eric Botcazou wrote: > Hi, > > this fixes the ICE during the build of the Ada runtime on the SPARC, a fallout > of the recent inliner changes: > http://gcc.gnu.org/ml/gcc-patches/2013-09/msg01033.html > > The ICE is triggered because the ldd peephole merges an ME

Re: [patch] tree-eh.c prototypes

2013-10-08 Thread Richard Biener
On Wed, Oct 2, 2013 at 7:54 PM, Andrew MacLeod wrote: > This patch moves the prototypes for tree-eh.c into a new file tree-eh.h. > This file is in fact really gimple-eh.. we'll rename that later with the > other tree->gimple renaming that is needed. > > however, using_eh_for_cleanups() is in fact

Re: [patch] The remainder of tree-flow.h refactored.

2013-10-08 Thread Richard Biener
On Fri, Oct 4, 2013 at 6:52 PM, Andrew MacLeod wrote: > This patch clears the rest of the improperly located prototypes out of > tree-flow.h. A bit larger than the last few, but I was pushing to clear > this up, and its not quite as bad as it seems :-) > > Of interest: > > * tree-flow.h now conta

Re: RFA: Add news item for ARC port contribution

2013-10-08 Thread Gerald Pfeifer
On Mon, 7 Oct 2013, Joern Rennecke wrote: OK to commit? Yes, this looks good to me, thanks! While you are at it, your entry in gcc/doc/contrib.texi could do with an update as well. :-) Gerald

Re: [patch] Add tree-ssa-loop.h and friends.

2013-10-08 Thread Andrew MacLeod
On 10/08/2013 05:57 AM, Richard Biener wrote: On Mon, Oct 7, 2013 at 3:39 PM, Andrew MacLeod wrote: On 10/07/2013 04:44 AM, Richard Biener wrote: On Thu, Oct 3, 2013 at 4:11 AM, Andrew MacLeod wrote: this patch consolidates tree-ssa-loop*.c files with new .h files as required (8 in total) A

Re: [patch] The remainder of tree-flow.h refactored.

2013-10-08 Thread Andrew MacLeod
On 10/08/2013 06:22 AM, Richard Biener wrote: On Fri, Oct 4, 2013 at 6:52 PM, Andrew MacLeod wrote: This patch clears the rest of the improperly located prototypes out of tree-flow.h. A bit larger than the last few, but I was pushing to clear this up, and its not quite as bad as it seems :-)

[patch] fix libstdc++/58659

2013-10-08 Thread Jonathan Wakely
PR libstdc++/58659 * include/bits/shared_ptr_base.h (__shared_count::__shared_count(P,D)): Delegate to constructor taking allocator. (__shared_count::_S_create_from_up): Inline into ... (__shared_count::__shared_count(unique_ptr&&): Here. Use std::con

Re: [patch] Add tree-ssa-loop.h and friends.

2013-10-08 Thread Andrew MacLeod
On 10/08/2013 07:38 AM, Andrew MacLeod wrote: On 10/08/2013 05:57 AM, Richard Biener wrote: Hm. Index: loop-iv.c === *** loop-iv.c (revision 203243) --- loop-iv.c (working copy) *** along with GCC; see the file COPY

Re: [patch] Add tree-ssa-loop.h and friends.

2013-10-08 Thread Andrew MacLeod
On 10/08/2013 08:35 AM, Andrew MacLeod wrote: On 10/08/2013 07:38 AM, Andrew MacLeod wrote: On 10/08/2013 05:57 AM, Richard Biener wrote: Hm. Index: loop-iv.c === *** loop-iv.c (revision 203243) --- loop-iv.c (working copy) **

Re: [patch] Add tree-ssa-loop.h and friends.

2013-10-08 Thread Richard Biener
On Tue, Oct 8, 2013 at 2:58 PM, Andrew MacLeod wrote: > On 10/08/2013 08:35 AM, Andrew MacLeod wrote: >> >> On 10/08/2013 07:38 AM, Andrew MacLeod wrote: >>> >>> On 10/08/2013 05:57 AM, Richard Biener wrote: Hm. Index: loop-iv.c

Re: [patch] fix libstdc++/58659

2013-10-08 Thread Jonathan Wakely
On 8 October 2013 13:33, Jonathan Wakely wrote: > PR libstdc++/58659 > * include/bits/shared_ptr_base.h > (__shared_count::__shared_count(P,D)): > Delegate to constructor taking allocator. > (__shared_count::_S_create_from_up): Inline into ... > (__shared_co

Fix ARM/Thumb non-interworking problem in libgcc

2013-10-08 Thread Jonathan Larmour
This is forwarded from PR58660 (). Please CC me on replies as I am not on the list. In doing some testing of GCC 4.7.3 on a target with an ARMv4T CPU (e.g. ARM7T or ARM9) I encountered a problem if using Thumb mode with interworking disabled. For

[patch] Tweak some libstdc++ tests.

2013-10-08 Thread Jonathan Wakely
2013-10-08 Jonathan Wakely * testsuite/*: Remove stray semi-colons after function definitions. Tested x86_64-linux, committed to trunk. commit 2c5f00a242d9aaf12cfcad0d217c4cad5b25b711 Author: Jonathan Wakely Date: Tue Oct 8 14:16:59 2013 +0100 * testsuite/*: Remove stray se

Re: [Patch] Fix the testcases that use bind_pic_locally

2013-10-08 Thread Vidya Praveen
On Tue, Oct 08, 2013 at 10:30:22AM +0100, Jakub Jelinek wrote: > On Tue, Oct 08, 2013 at 10:14:59AM +0100, Vidya Praveen wrote: > > There are several tests that use "dg-add-options bind_pic_locally" in order > > to > > add -fPIE or -fpie when -fPIC or -fpic are used respectively with the > > expe

Re: [patch] Add tree-ssa-loop.h and friends.

2013-10-08 Thread Andrew MacLeod
On 10/08/2013 09:18 AM, Richard Biener wrote: On Tue, Oct 8, 2013 at 2:58 PM, Andrew MacLeod wrote: I just took a quick stab at it... I think its pretty involved and someone with better loop comprehension should probably look at the followup of removing that requirement. estimate_numbers_of_

Re: [PATCH v2] Fix libgfortran cross compile configury w.r.t newlib

2013-10-08 Thread Marcus Shawcroft
On 1 October 2013 12:40, Marcus Shawcroft wrote: > Patch attached. > > /Marcus > > 2013-10-01 Marcus Shawcroft > > * configure.ac (AC_CHECK_FUNCS_ONCE): Add for exit() then make > existing AC_CHECK_FUNCS_ONCE dependent on outcome. Ping.

[gomp4] Adjust some gcc.dg/autopar/ tests

2013-10-08 Thread Jakub Jelinek
Hi! These tests were expecting 5 loopfn matches, 3 on the fn definition, one as GOMP_parallel_start argument and one called in between GOMP_parallel_start and GOMP_parallel_end. But the new API is to call GOMP_parallel with the function and not call the outlined function nor GOMP_parallel_end dir

[gomp4] Small libgomp testsuite tweaks

2013-10-08 Thread Jakub Jelinek
Hi! I've noticed that udr-8.C failed on i686-linux, apparently because i was uninitialized. udr-3.c had the same bug, but didn't FAIL because of that on either x86_64-linux nor i686-linux, and udr-2.c had just unused variable. 2013-10-08 Jakub Jelinek * testsuite/libgomp.c/udr-2.c (m

[PATCH][AArch64] Vneg NEON intrinsics modified

2013-10-08 Thread Alex Velenko
Hi, This patch implements the behavior of the following neon intrinsics using C: vneg[q]_f[32,64] vneg[q]_s[8,16,32,64] Regression tests for listed intrinsics included. I ran a full regression test for aarch64-none-elf with no regressions. Ok? Thanks, Alex gcc/testsuite/ 2013-10-08 Alex Vel

Re: [C++ Patch] PR 58568

2013-10-08 Thread Jason Merrill
OK. Jason

[PATCH][AARCH64] Vdiv NEON intrinsic

2013-10-08 Thread Alex Velenko
Hi, This patch implements the behavior of vdiv_f64 intrinsic and adds regression tests for vdiv[q]_f[32,64] NEON intrinsics. Full aarch64-none-elf regression test ran with no regressions. Is it OK? Thanks, Alex gcc/testsuite/ 2013-09-10 Alex Velenko * gcc.target/aarch64/vdiv_

Re: [PATCH] alternative hirate for builtin_expert

2013-10-08 Thread Dehao Chen
Thanks for applying the patch. Backported to google-4_8 I still have some concern when inlining .part function into its original function: basically, the gimple_block for that call may be NULL, but it does not make sense to clear all block info for all stmts in the .part function. Dehao On Tue,

[PATCH][AArch64] NEON vadd_f64 and vsub_f64 intrinsics modified

2013-10-08 Thread Alex Velenko
Hi, This patch implements the behavior of vadd_f64 and vsub_f64 NEON intrinsics. Regression tests are added. Regression tests for aarch64-none-elf completed with no regressions. OK? Thanks, Alex gcc/testsuite/ 2013-10-08 Alex Velenko * gcc.target/aarch64/vadd_f64.c: New testcase.

[PATCH][AArch64] NEON vclz intrinsic modified

2013-10-08 Thread Alex Velenko
Hi, This patch implements the behavior and regression test for NEON intrinsics vclz[q]_[s,u][8,16,32] No problems found when running aarch64-none-elf regressions tests. Is patch OK? Thanks, Alex gcc/testsuite/ 2013-10-08 Alex Velenko * gcc.target/aarch64/vclz.c: New testcase.

Re: [patch] Fix PR middle-end/58570

2013-10-08 Thread Eric Botcazou
> Probably because the actual accesses may overlap if we choose to > perform a bigger access. Nope, simply because they share a byte. > The same can happen if we for struct { char c1; char c2; } perform > an HImode access in case the target doesn't support QImode accesses. > Basically anytime we

Re: [c++-concepts] friends regression

2013-10-08 Thread Andrew Sutton
No, any current_template_reqs are reset (set to null) before parsing any trailing requirements and restored after the fact. Andrew Sutton On Mon, Oct 7, 2013 at 3:05 PM, Jason Merrill wrote: > OK. > > If we have a friend declaration inside a constrained partial specialization, > will that still

[gomp4] Fix task cancellation

2013-10-08 Thread Jakub Jelinek
Hi! I've noticed that occassionally on larger box under high load cancel-taskgroup-{2.c,2.C,3.C} tests would time out. The problem is that sometimes gomp_team_barrier_clear_task_pending wasn't called, when GOMP_taskgroup_end (or could be GOMP_taskwait) ate the last few threads. We have task_count

[PATCH]: Fix PR58542, Arguments of __atomic_* functions are converted in unsigned mode

2013-10-08 Thread Uros Bizjak
Hello! As shown in the attached testcase, arguments of various __atomic builtins should be converted as signed, so the immediates get properly extended. 2013-10-08 Uros Bizjak * optabs.c (maybe_emit_atomic_exchange): Convert operands as signed. (maybe_emit_sync_lock_test_and_set): Dit

[gomp4] Fix thread-limit-{1,2}.c tests

2013-10-08 Thread Jakub Jelinek
Hi! The last probelm I've occassionally saw FAILing under high load was a thinko in thread-limit-1.c test. The test meant to verify that no more than omp_get_thread_limit () total threads are running in the same contention group, but foolishly assumed that all the nested GOMP_parallels will happe

Re: New attribute: returns_nonnull

2013-10-08 Thread Jeff Law
On 10/07/13 08:17, Marc Glisse wrote: Hello, this patch adds an attribute to let the compiler know that a function never returns NULL. I saw some ECF_* flags, but the attribute seems sufficient. I considered using nonnull(0), but then it would have been confusing that the version of nonnull with

[C++ Patch] PR 58633

2013-10-08 Thread Paolo Carlini
Hi, in this ICE on valid, 4.7/4.8/4.9 Regression, the ICE happens in the second half of cp_parser_decltype_expr, when cp_parser_abort_tentative_parse is called in an inconstent status: tentative parsing is committed and no errors. The reason is the following: in its main loop cp_parser_postfi

Re: [PATCH 2/6] Andes nds32: machine description of nds32 porting (1).

2013-10-08 Thread Richard Sandiford
Chung-Ju Wu writes: > On 10/6/13 5:36 PM, Richard Sandiford wrote: >> Thanks for the updates. >> >> Chung-Ju Wu writes: >>> >>> Now we remove all "use"s and "clobber"s from parallel rtx and >>> use predicate function to check stack push/pop operation. >>> Furthermore, once I remove unspec rtx as

Re: New attribute: returns_nonnull

2013-10-08 Thread Marc Glisse
On Tue, 8 Oct 2013, Jeff Law wrote: On 10/07/13 08:17, Marc Glisse wrote: Hello, this patch adds an attribute to let the compiler know that a function never returns NULL. I saw some ECF_* flags, but the attribute seems sufficient. I considered using nonnull(0), but then it would have been conf

[0/6] Merge of gomp-4_0-branch to trunk

2013-10-08 Thread Jakub Jelinek
Hi! I'd like to merge (most of) gomp-4_0-branch to trunk. Except for unknown bugs and known outstanding unclear spots in the standard, the branch right now implements OpenMP 4.0 standard with the following caveats that I hope can be dealt with incrementally later on: 1) Fortran front-end support o

Re: [1/6] OpenMP 4.0 C FE support

2013-10-08 Thread Jakub Jelinek
Hi! Sorry for the subject, that was meant to be [2/6], not [1/6]. Jakub

Re: Cleanup patches

2013-10-08 Thread Jakub Jelinek
On Tue, Oct 08, 2013 at 09:17:35AM +0200, Thomas Schwinge wrote: > Here are a few cleanup patches, mostly in the realm of OpenMP, so Jakub > gets a CC. OK to commit? They look ok to me, but I'd prefer if they could go in after the merge I've just posted. Jakub

Re: New attribute: returns_nonnull

2013-10-08 Thread Jeff Law
On 10/08/13 13:41, Marc Glisse wrote: Glad to see you checked this and have a test for it. I am slowly starting to understand how reviewers think ;-) That's a huge part of the submission process. Once you know what folks are looking for the path to approval gets appropriately short :-)

[4/6] OpenMP 4.0 gcc testsuite

2013-10-08 Thread Jakub Jelinek
Hi! Apparently the 3/6 and 4/6 patches didn't make it through to gcc-patches, although they aren't the largest. Trying to post them bzip2ed now if I get more luck to get it through. 2013-10-08 Jakub Jelinek gcc/testsuite/ * c-c++-common/gomp/atomic-15.c: Adjust for C diagnostics.

Patch: Add #pragma ivdep support to the ME and C FE (was: Re: RFC patch for #pragma ivdep)

2013-10-08 Thread Tobias Burnus
Jakub Jelinek wrote: On Tue, Oct 08, 2013 at 08:51:50AM +0200, Tobias Burnus wrote: + if (loop->latch && loop->latch->next_bb != EXIT_BLOCK_PTR + && bb_seq_addr (loop->latch->next_bb)) Why this bb_seq_addr guard? Without, I get a segfault in the stage 1 (prev-gcc/xg++) compiler

Re: [PATCH, PR 57748] Check for out of bounds access, Part 2

2013-10-08 Thread Eric Botcazou
> I agree, that assigning a non-BLKmode to structures with zero-sized arrays > should be considered a bug. Fine, then let's apply Martin's patch, on mainline at least. > And again, this is not only a problem of structures with zero-sized > arrays at the end. Remember my previous example code: > O

Add a param to decide stack slot sharing at -O0

2013-10-08 Thread Easwaran Raman
In cfgexpand.c, variables in non-overlapping lexical scopes are assigned same stack locations at -O1 and above. At -O0, this is attempted only if the size of the stack objects is above a threshold (32). The rationale is at -O0, more variables are going to be in the stack and the O(n^2) stack slot s

[Patch, Fortran] Use ANNOTATE_EXPR annot_expr_ivdep_kind for DO CONCURRENT

2013-10-08 Thread Tobias Burnus
This patch requires my pending ME and C FE patch: http://gcc.gnu.org/ml/gcc-patches/2013-10/msg00514.html Using C/C++'s #pragma ivdep or – with the attached Fortran patch – "do concurrent", the loop condition is annotated such that later the loop's vectorization safelen is set to infinity (wel

Re: Patch: Add #pragma ivdep support to the ME and C FE (was: Re: RFC patch for #pragma ivdep)

2013-10-08 Thread Jakub Jelinek
On Tue, Oct 08, 2013 at 10:51:29PM +0200, Tobias Burnus wrote: > + return false; > + } > + c_parser_for_statement (parser, true); > + return false; > + > case PRAGMA_GCC_PCH_PREPROCESS: >c_parser_error (parser, "%<#pragma GCC pch_preprocess%> must be > first"); >

Re: [PATCH] Enhance phiopt to handle BIT_AND_EXPR

2013-10-08 Thread Jeff Law
On 09/30/13 09:57, Andrew Pinski wrote: On Mon, Sep 30, 2013 at 2:29 AM, Zhenqiang Chen wrote: Hi, The patch enhances phiopt to handle cases like: if (a == 0 && (...)) return 0; return a; Boot strap and no make check regression on X86-64 and ARM. Is it OK for trunk? From someo

Re: [C++ Patch] PR 58448

2013-10-08 Thread Paolo Carlini
On 10/04/2013 02:04 PM, Paolo Carlini wrote: ... and this is a more straightforward approach. Also tested x86_64-linux. I reverted this for now. Was causing problems (c++/58665). Thanks, Paolo.

RE: [PATCH, PR 57748] Check for out of bounds access, Part 2

2013-10-08 Thread Bernd Edlinger
Hi, On Tue, 8 Oct 2013 22:50:21, Eric Botcazou wrote: > >> I agree, that assigning a non-BLKmode to structures with zero-sized arrays >> should be considered a bug. > > Fine, then let's apply Martin's patch, on mainline at least. > That would definitely be a good move. Maybe someone should approv

Re: [C++ Patch] PR 58633

2013-10-08 Thread Paolo Carlini
.. a curiosity: the cp_parser_commit_to_tentative_parse at the end of cp_parser_pseudo_destructor_name, which didn't exist in 4.6.x and we can consider the root of this issue, is also my fault: http://gcc.gnu.org/ml/gcc-patches/2011-05/msg02246.html From a different angle, I'm happy of the

Re: [patch] The remainder of tree-flow.h refactored.

2013-10-08 Thread Andrew MacLeod
On 10/08/2013 07:44 AM, Andrew MacLeod wrote: On 10/08/2013 06:22 AM, Richard Biener wrote: graphite.h should be unnecessary with moving the pass struct like you did for other loop opts. Likewise tree-parloops.h (well, ok, maybe you need parallelized_function_p, even though it's implementation

Re: Enable SSE math on i386 with -Ofast

2013-10-08 Thread Jan Hubicka
Hi, this is patch I ended up comitting after some furhter testing. The difference to initial version is that it now eanbles SSE math with -ffast-math too and it does so outside the ugly target macro. Bootstrapped/regtested x86_64-linux, tested with -m32 Honza * config/i386/i386.c (ix86

Go patch committed: Error for qualified ID in struct composite lit

2013-10-08 Thread Ian Lance Taylor
This patch to the Go frontend gives an error if a qualified identifier is used as a field name in a struct composite literal. This is trickier than one might expect since qualified identifiers are fine as keys in a slice or map composite literal. Bootstrapped and ran Go testsuite on x86_64-unknow

libgo patch committed: Do not report thunks/recover in backtrace

2013-10-08 Thread Ian Lance Taylor
This patch to libgo avoids returning thunk or recover functions in a backtrace or when calling runtime.Caller or friends. This is to give a stack backtrace more like that generated by the gc compiler. In particular, it is so that runtime.Caller(n) will return stack trace that more closely corresp

rl78: fix %c modifier

2013-10-08 Thread DJ Delorie
namespaces are so small... committed. * config/rl78/rl78.c (rl78_print_operand_1): Change %c to %C to avoid conflict with the MI use of %c. * config/rl78/rl78-real.md: change %c to %C throughout. * config/rl78/rl78-virt.md: Likewise. Index: config/rl78/rl78-real.m

RE: [PATCH] reimplement -fstrict-volatile-bitfields v4, part 2/2

2013-10-08 Thread Bernd Edlinger
Hi, On Mon, 30 Sep 2013 16:18:30, DJ Delorie wrote: > > As per my previous comments on this patch, I will not approve the > changes to the m32c backend, as they will cause real bugs in real > hardware, and violate the hardware's ABI. The user may use > -fno-strict-volatile-bitfields if they do not

[rl78] tweaks to moviqi/movsi expander

2013-10-08 Thread DJ Delorie
Minor fix. Committed. * config/rl78/rl78-expand.md (movqi): use operands[] not operandN. (movhi): Likewise. 2013-10-08 Jan Hubicka Index: config/rl78/rl78-expand.md === --- config/rl78/rl78-expand.md (revision

Re: Add a param to decide stack slot sharing at -O0

2013-10-08 Thread Andi Kleen
Easwaran Raman writes: > In cfgexpand.c, variables in non-overlapping lexical scopes are > assigned same stack locations at -O1 and above. At -O0, this is > attempted only if the size of the stack objects is above a threshold > (32). The rationale is at -O0, more variables are going to be in the

Re: [SKETCH] Refactor implicit function template implementation and fix 58534, 58536, 58548, 58549 and 58637.

2013-10-08 Thread Jason Merrill
On 10/07/2013 05:14 AM, Adam Butcher wrote: + /* Forbid ambiguous implicit pack expansions by only allowing +a single generic type in such a parameter. +XXX: Maybe allow if explicitly specified template +XXX: 'typename..

RE: [PATCH] Enhance phiopt to handle BIT_AND_EXPR

2013-10-08 Thread Zhenqiang Chen
> -Original Message- > From: Jeff Law [mailto:l...@redhat.com] > Sent: Wednesday, October 09, 2013 5:00 AM > To: Andrew Pinski; Zhenqiang Chen > Cc: GCC Patches > Subject: Re: [PATCH] Enhance phiopt to handle BIT_AND_EXPR > > On 09/30/13 09:57, Andrew Pinski wrote: > > On Mon, Sep 30, 201