Re: [google/gcc-4_9] Add -ftwo-level-line-tables and -gline-tables-only options

2015-01-28 Thread Dehao Chen
On Wed, Jan 28, 2015 at 3:04 PM, Cary Coutant wrote: > > >> +static subprog_entry * > >> +add_subprog_entry (tree decl, bool is_inlined) > >> +{ > >> + subprog_entry **slot; > >> + subprog_entry *entry; > >> + > >> + slot = subprog_table->find_slot_with_hash (decl, DECL_UID (decl), > >> INSERT

Re: [google/gcc-4_9] Add -ftwo-level-line-tables and -gline-tables-only options

2015-01-28 Thread Cary Coutant
>> > Not quite clear why we need block_table. This table is not gonna be >> > emitted. And we can easily get subprog_entry through block->block_num >> >> When final_scan_insn() calls dwarf2out_begin_block(), all it passes is >> a block number. I don't know a way to get from block number to the >> b

Re: [google/gcc-4_9] Add -ftwo-level-line-tables and -gline-tables-only options

2015-01-28 Thread Dehao Chen
On Wed, Jan 28, 2015 at 4:34 PM, Cary Coutant wrote: >>> > Not quite clear why we need block_table. This table is not gonna be >>> > emitted. And we can easily get subprog_entry through block->block_num >>> >>> When final_scan_insn() calls dwarf2out_begin_block(), all it passes is >>> a block numb

Re: [wwwdocs] Mention ipa-ICF and auto-FDO

2015-01-28 Thread Gerald Pfeifer
On Monday 2015-01-26 07:10, Jan Hubicka wrote: > Index: changes.html > === > + An Identical Code Folding (ICF) pass (controled via controlled > + this pass unifies about 29000 functions that is 10% overall. "functions, that

RE: [PATCH] [gcc, combine] PR46164: Don't combine the insns if a volatile register is contained.

2015-01-28 Thread Hale Wang
Hi Segher, I have updated the patch as you suggested. Both the patch and the changelog are attached. By the way, the test case provided by Tim Pambor in PR46164 was a different bug with PR46164. So I resubmitted the bug in https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64818. And this patch is just

Re: Fix tls model dumping

2015-01-28 Thread Alan Modra
On Wed, Jan 28, 2015 at 08:56:22PM +0100, Jan Hubicka wrote: > -const char * const tls_model_names[]={"none", "tls-emulated", "tls-real", > - "tls-global-dynamic", "tls-local-dynamic", > - "tls-initial-exec", "tls-local-exec"}; > +

RE: [Patch][wwwdocs]Deprecate the ARM TPCS related options in gcc 5.0

2015-01-28 Thread Terry Guo
> -Original Message- > From: Gerald Pfeifer [mailto:ger...@pfeifer.com] > Sent: Thursday, January 29, 2015 2:53 AM > To: Terry Guo > Cc: gcc-patches@gcc.gnu.org; Richard Earnshaw; Ramana Radhakrishnan > Subject: RE: [Patch][wwwdocs]Deprecate the ARM TPCS related options in > gcc 5.0 > >

Re: Fix tls model dumping

2015-01-28 Thread Jan Hubicka
> On Wed, Jan 28, 2015 at 08:56:22PM +0100, Jan Hubicka wrote: > > -const char * const tls_model_names[]={"none", "tls-emulated", "tls-real", > > - "tls-global-dynamic", "tls-local-dynamic", > > - "tls-initial-exec", "tls-local-exec"};

Fix flag_fp_contract_mode with LTO

2015-01-28 Thread Jan Hubicka
Hi, Igor found an performance regression related to my patch enabling option streaming. It turns out that with LTO we disable FMA instruction generation because fp_contract_mode is not streamed and set to 0. This is because patch https://gcc.gnu.org/ml/gcc-patches/2010-11/msg01827.html that makes f

Re: Fix tls model dumping

2015-01-28 Thread Alan Modra
On Thu, Jan 29, 2015 at 06:07:16AM +0100, Jan Hubicka wrote: > > On Wed, Jan 28, 2015 at 08:56:22PM +0100, Jan Hubicka wrote: > > > -const char * const tls_model_names[]={"none", "tls-emulated", "tls-real", > > > - "tls-global-dynamic", "tls-local-dynamic", > > > -

Re: [PATCH] PR64635 - load libgomp-plugin-host_nonshm shared library with correct suffix

2015-01-28 Thread Thomas Schwinge
Hi! First, thanks for improving portability of the libgomp plugin interface! On Wed, 28 Jan 2015 16:43:19 -0500, Jack Howarth wrote: > There is one other issue that I have been > pondering about filing a PR. In fink and MacPorts, FSF gcc is built > and packaged using either... > > --prefix=/s

ICF versus inlining

2015-01-28 Thread Jan Hubicka
Hi, the PR is about function not being inlined because it is called via a wrapper introduced by ICF merging code. cgraph_node::create_wrapper set call_stmt_cannot_inline_p that I suggested to Martin to try to figure out how much of merging is undone by inliner. It was not meant to get into mainlin

Re: [libobjc] Fix failures on AIX (PR libobjc/63765)

2015-01-28 Thread Alexandre Oliva
On Jan 28, 2015, Mike Stump wrote: > On Jan 28, 2015, at 2:27 AM, Rainer Orth > wrote: >> There are two ways to fix this: >> >> * Remove the definition of _XOPEN_SOURCE completely. This is slightly >> more risky, but more future-proof since defining features test macros >> has been an endless

Re: [PATCH] Fix scheduling undeterminism from sorting with DEBUG_INSNs

2015-01-28 Thread Alexandre Oliva
On Jan 19, 2015, Maxim Kuvyrkov wrote: > Presence or absence of DEBUG_INSNs in the ready list can change the > comparison order, and cause slightly different instruction schedules. > The solution that I propose (and that the patch implements) is to sort > DEBUG_INSNs separately from normal insns

Re: [PATCH] Fix scheduling undeterminism from sorting with DEBUG_INSNs

2015-01-28 Thread Maxim Kuvyrkov
On Jan 29, 2015, at 10:36 AM, Alexandre Oliva wrote: > On Jan 19, 2015, Maxim Kuvyrkov wrote: > >> Presence or absence of DEBUG_INSNs in the ready list can change the >> comparison order, and cause slightly different instruction schedules. > >> The solution that I propose (and that the patch i

<    1   2