Re: [patch,avr]: Fix various problems with specs and specs file generation.

2015-03-02 Thread Denis Chertykov
2015-03-02 22:40 GMT+03:00 Georg-Johann Lay : > The new specs file generation introduces several problems. This patch > > - Fix build warnings > > - Clean up unused code and the old, now dead specs definitions. > > - Removes unused files and adjust build scripts / rules. > > - Issues with avr-libc

[PING][PATCH] ASan phase place change

2015-03-02 Thread Marat Zakirov
On 02/24/2015 03:49 PM, Marat Zakirov wrote: On 02/20/2015 03:07 PM, Jakub Jelinek wrote: On Fri, Feb 20, 2015 at 02:59:51PM +0300, Marat Zakirov wrote: Here is simple patch that moves asan phase just behind sanopt for all O0/O1/O2/O3 modes which gives +7% on x86 SPEC2006 (ref dataset). Regre

Re: Strengthen ICF hash

2015-03-02 Thread Jan Hubicka
Andi, > Jan Hubicka writes: > > > > The hash itself is quite simple (borrowing incremental hash for constants > > adding > > very simple match for other stuff + logic to skip things that may match > > even if > > they are syntactticaly different). The hash can be strenghtened > > significantly,

[PATCH] Turn RETURN_ADDR_IN_PREVIOUS_FRAME into C expression

2015-03-02 Thread Max Filippov
This allows a target to support both windowed and non-windowed ABI. 2015-03-03 Max Filippov gcc/ * builtins.c (expand_builtin_return_addr): Add RETURN_ADDR_IN_PREVIOUS_FRAME to 'if' condition, remove surrounding #ifdef. * config/sparc/sparc.h (RETURN_ADDR_IN_PRE

RE: [PATCH] Fix removing of df problem in df_finish_pass

2015-03-02 Thread Thomas Preud'homme
> From: Bernhard Reutner-Fischer [mailto:rep.dot@gmail.com] > Sent: Saturday, February 28, 2015 4:00 AM > > use df_remove_problem rather than manually removing problems, > living > > leaving Indeed. Please find updated changelog below: 2015-03-03 Thomas Preud'homme * df-core.c

Re: [PATCH][ARM]Automatically add -mthumb for thumb-only target when mode isn't specified

2015-03-02 Thread Terry Guo
On Mon, Mar 2, 2015 at 9:08 PM, Maxim Kuvyrkov wrote: >> On Mar 2, 2015, at 4:44 AM, Terry Guo wrote: >> >> Hi there, >> >> If target mode isn't specified via either gcc configuration option >> --with-mode or command line, this patch intends to improve gcc driver to >> automatically add option -m

Re: [PATCH] PR 65138, Fix PowerPC little endian -flto bugs

2015-03-02 Thread David Edelsohn
On Mon, Mar 2, 2015 at 6:55 PM, Michael Meissner wrote: > This patch fixes PR 65138, which shows up when you configure a compiler on a > 64-bit PowerPC little endian system without using the --with-cpu=power8 > configuration option, and run tests involve -flto. > > The following tests now run when

Re: [PATCH, rs6000, testsuite] Changes for unaligned vector load/store support on POWER8

2015-03-02 Thread David Edelsohn
On Mon, Mar 2, 2015 at 4:37 PM, Bill Schmidt wrote: > Hi, > > On Thu, 2015-02-26 at 16:48 -0600, Bill Schmidt wrote: >> >> Except this didn't pass regression testing. I'll continue to poke at >> it. > > I figured out the issue here. My handling of the > TARGET_ALLOW_MOVMISALIGN and TARGET_EFFICI

RE: [Patch, MIPS] Fix PR 58158 - ICE on MIPS Loongson

2015-03-02 Thread Steve Ellcey
On Mon, 2015-03-02 at 15:54 -0800, Matthew Fortune wrote: > Thanks for looking through and catching this. > > I had conflicting thoughts on whether the new condition should > reference both !ISA_HAS_FP_CONDMOVE || ISA_HAS_SEL but if we take it > that FP_CONDMOVE is the only way to get an integer c

[PATCH] PR 65138, Fix PowerPC little endian -flto bugs

2015-03-02 Thread Michael Meissner
This patch fixes PR 65138, which shows up when you configure a compiler on a 64-bit PowerPC little endian system without using the --with-cpu=power8 configuration option, and run tests involve -flto. The following tests now run when I build a configuration without the --with-cpu=power8 option:

RE: [Patch, MIPS] Fix PR 58158 - ICE on MIPS Loongson

2015-03-02 Thread Matthew Fortune
Thanks for looking through and catching this. I had conflicting thoughts on whether the new condition should reference both !ISA_HAS_FP_CONDMOVE || ISA_HAS_SEL but if we take it that FP_CONDMOVE is the only way to get an integer conditional move based on an FP condition then that's fine. > 2015-0

[Patch, MIPS] Fix PR 58158 - ICE on MIPS Loongson

2015-03-02 Thread Steve Ellcey
While looking at the outstanding GCC regressions in bugzilla I noticed PR 58158 which has a proposed fix in the comments but which doesn't seem to have ever been checked in. I updated the change to apply to top-of-tree sources and tested the specific test case but I don't have a Loongson system to

Re: ipa-icf::merge TLC

2015-03-02 Thread Christophe Lyon
On 2 March 2015 at 21:21, Jan Hubicka wrote: >> >> >> On 01/03/15 16:47, Christophe Lyon wrote: >> >On 27 February 2015 at 21:49, Jan Hubicka wrote: >> >>> >> >>>../sysdeps/gnu/siglist.c:72:1: internal compiler error: in >> >>>address_matters_p, at symtab.c:1908 >> >>> versioned_symbol (libc, _

Re: [patch] PR rtl-optimization/65220: avoid integer division in stack alignment

2015-03-02 Thread Richard Henderson
On 03/02/2015 10:55 AM, Aldy Hernandez wrote: > >>> Do you actually need the reload completed? Couldn't this be legitimately >>> split >>> before reload (and then parts of it DCE'd as necessary)? >>> >>> Otherwise, the split condition does need "&&" as mentioned by Uros. >> >> Oh, I forgot -- ev

Re: [PATCH, rs6000, testsuite] Changes for unaligned vector load/store support on POWER8

2015-03-02 Thread Bill Schmidt
Hi, On Thu, 2015-02-26 at 16:48 -0600, Bill Schmidt wrote: > > Except this didn't pass regression testing. I'll continue to poke at > it. I figured out the issue here. My handling of the TARGET_ALLOW_MOVMISALIGN and TARGET_EFFICIENT_UNALIGNED_VSX macros was correct, but there was another probl

Re: [RFC 1/2] Turn RETURN_ADDR_IN_PREVIOUS_FRAME into C expression

2015-03-02 Thread David Miller
From: Max Filippov Date: Sun, 1 Mar 2015 09:34:38 +0300 > Richard, David, Eric, > > could you please take a look and possibly approve the below changes for > sparc? I don't have any objection to the sparc changes.

Re: [PATCH] fix URL

2015-03-02 Thread Sandra Loosemore
On 03/02/2015 11:20 AM, Ulrich Drepper wrote: A trivial patch to fix the URL of my paper referenced in the documentation. The RH server isn't the canonical address even though it will automatically redirect to the correct address. At least for now, who knows. diff --git a/gcc/doc/invoke.texi

Avoid cycles in the inline plan

2015-03-02 Thread Jan Hubicka
Hi, in the testcase bellow we manage to produce cycle in inline plan because there is indirectly recrusive function that appears called once. I am having problems with the testcase. The second file outht to be compiled with -O2, but it is not. Is the dg-options in the LTO testsuite broken now o

Re: ipa-icf::merge TLC

2015-03-02 Thread Jan Hubicka
> > > On 01/03/15 16:47, Christophe Lyon wrote: > >On 27 February 2015 at 21:49, Jan Hubicka wrote: > >>> > >>>../sysdeps/gnu/siglist.c:72:1: internal compiler error: in > >>>address_matters_p, at symtab.c:1908 > >>> versioned_symbol (libc, __new_sys_sigabbrev, sys_sigabbrev, GLIBC_2_3_3); > >

Re: [gofrontend-dev] Re: Go patch committed: Don't strip Go programs

2015-03-02 Thread Lynn A. Boger
libgo sizes 32665042: original size 27607770: after removing the debug sections using the objcopy command from below 22474602: after removing all debug sections using objcopy On 03/02/2015 02:08 PM, Ian Lance Taylor wrote: On Mon, Mar 2, 2015 at 12:05 PM, Lynn A. Boger wrote: I removed

Do not dissolve comdat groups in inliner

2015-03-02 Thread Jan Hubicka
Hi, in the testcase bellow the comdat local function is inlined and inliner mistakely dissolve the whole comdat group. This is wrong and leads to ICE here, but it also may lead to undefined symbols due to groups being wrong. Bootstrapped/regtested x86_64-linux, comitted. PR ipa/64988

Re: [gofrontend-dev] Re: Go patch committed: Don't strip Go programs

2015-03-02 Thread Lynn A. Boger
I removed the 3 debug sections from libgo that were listed on the objcopy command. I left in: debug_info, debug_line, debug_abbrev, debug_ranges, and debug_str in their original form. On 03/02/2015 01:58 PM, Ian Lance Taylor wrote: On Mon, Mar 2, 2015 at 11:50 AM, Lynn A. Boger wrote: obj

Re: [gofrontend-dev] Re: Go patch committed: Don't strip Go programs

2015-03-02 Thread Ian Lance Taylor
On Mon, Mar 2, 2015 at 12:05 PM, Lynn A. Boger wrote: > I removed the 3 debug sections from libgo that were listed on the objcopy > command. > > I left in: debug_info, debug_line, debug_abbrev, debug_ranges, and > debug_str in their original form. Understood. I mean: if you add up the total siz

Re: [gofrontend-dev] Re: Go patch committed: Don't strip Go programs

2015-03-02 Thread Ian Lance Taylor
On Mon, Mar 2, 2015 at 11:50 AM, Lynn A. Boger wrote: > > objcopy -R .debug_aranges -R .debug_frame -R .debug_loc libgo.so.7.0.0 What percentage of the total size of .debug sections got removed by this process? A problem with that approach is that .debug_info and .debug_abbrevs sections are larg

Re: [gofrontend-dev] Re: Go patch committed: Don't strip Go programs

2015-03-02 Thread Lynn A. Boger
On 02/27/2015 02:46 PM, Ian Lance Taylor wrote: On Fri, Feb 27, 2015 at 12:07 PM, Matthias Klose wrote: is there anything which could be stripped without scarifying functionality? Linux distributions usually strip things by default, so a hint what exactly is needed to keep the functionality wo

[patch,avr]: Fix various problems with specs and specs file generation.

2015-03-02 Thread Georg-Johann Lay
The new specs file generation introduces several problems. This patch - Fix build warnings - Clean up unused code and the old, now dead specs definitions. - Removes unused files and adjust build scripts / rules. - Issues with avr-libc awareness: -- Makes specs-generation aware of avr-libc (i

Re: ipa-icf::merge TLC

2015-03-02 Thread Alex Velenko
On 01/03/15 16:47, Christophe Lyon wrote: On 27 February 2015 at 21:49, Jan Hubicka wrote: ../sysdeps/gnu/siglist.c:72:1: internal compiler error: in address_matters_p, at symtab.c:1908 versioned_symbol (libc, __new_sys_sigabbrev, sys_sigabbrev, GLIBC_2_3_3); ^ 0x66a080 symtab_node::add

Re: [PATCH] Fix PR65261

2015-03-02 Thread Markus Trippelsdorf
On 2015.03.02 at 18:53 +0100, Marek Polacek wrote: > On Mon, Mar 02, 2015 at 06:22:49PM +0100, Markus Trippelsdorf wrote: > > --- a/libcpp/lex.c > > +++ b/libcpp/lex.c > > @@ -519,6 +519,7 @@ init_vectorized_lexer (void) > > and VSX unaligned loads (when VSX is available). This is otherwise >

Re: [patch] PR rtl-optimization/65220: avoid integer division in stack alignment

2015-03-02 Thread Aldy Hernandez
Do you actually need the reload completed? Couldn't this be legitimately split before reload (and then parts of it DCE'd as necessary)? Otherwise, the split condition does need "&&" as mentioned by Uros. Oh, I forgot -- even if you can exclude reload_completed, you'd then need to use "&& 1"

[patch] PR lto/65276 remove odr_violated assert

2015-03-02 Thread Aldy Hernandez
TYPE_BINFO is null when no optimization is used, but odr_violated is unset. Fixed and approved in the PR by Honza. Tested on x86-64 Linux. Committed to mainline. commit a26ff3a2376a898bef8ca8ab17a457052d677a62 Author: Aldy Hernandez Date: Mon Mar 2 09:55:28 2015 -0800 PR lto/65276

Re: [patch] PR rtl-optimization/65220: avoid integer division in stack alignment

2015-03-02 Thread Richard Henderson
On 03/02/2015 09:55 AM, Richard Henderson wrote: > On 02/26/2015 05:46 PM, Aldy Hernandez wrote: >> +;; Optimize division or modulo by constant power of 2, if the constant >> +;; materializes only after expansion. >> +(define_insn_and_split "*udivmod4_pow2" >> + [(set (match_operand:SWI48 0 "regis

Re: [patch] PR rtl-optimization/65220: avoid integer division in stack alignment

2015-03-02 Thread Richard Henderson
On 03/02/2015 10:25 AM, Aldy Hernandez wrote: > On 03/02/2015 09:55 AM, Richard Henderson wrote: >> On 02/26/2015 05:46 PM, Aldy Hernandez wrote: >>> +;; Optimize division or modulo by constant power of 2, if the constant >>> +;; materializes only after expansion. >>> +(define_insn_and_split "*udiv

Re: [patch] PR rtl-optimization/65220: avoid integer division in stack alignment

2015-03-02 Thread Aldy Hernandez
On 03/02/2015 09:55 AM, Richard Henderson wrote: On 02/26/2015 05:46 PM, Aldy Hernandez wrote: +;; Optimize division or modulo by constant power of 2, if the constant +;; materializes only after expansion. +(define_insn_and_split "*udivmod4_pow2" + [(set (match_operand:SWI48 0 "register_operand

Re: [PATCH, alpha]: Fix PR/47230 [4.6/4.7 Regression] gcc fails to bootstrap on alpha in stage2 with "relocation truncated to fit: GPREL16 against ..."

2015-03-02 Thread Richard Henderson
On 02/27/2015 05:11 AM, Uros Bizjak wrote: > On Wed, Feb 25, 2015 at 8:18 PM, Richard Henderson wrote: >> On 02/25/2015 09:02 AM, Uros Bizjak wrote: >>> The patch was tested on alpha-linux-gnu and alphaev68-linux-gnu for >>> all default languages plus obj-c++ and go. >>> >>> OK for mainline? >> >>

Re: [patch, avr] Take 2: Fix PR64331: insn output and insn length computation rely on REG_DEAD notes.

2015-03-02 Thread Georg-Johann Lay
Am 03/02/2015 um 05:12 PM schrieb Steven Bosscher: On Sat, Feb 28, 2015 at 5:38 PM, Georg-Johann Lay wrote: Am 02/26/2015 um 11:45 PM schrieb Steven Bosscher: On Thu, Feb 26, 2015 at 8:35 PM, Georg-Johann Lay wrote: Take #2 introduces a new, avr-specific rtl pass whose sole purpose is to rec

[PATCH] fix URL

2015-03-02 Thread Ulrich Drepper
A trivial patch to fix the URL of my paper referenced in the documentation. The RH server isn't the canonical address even though it will automatically redirect to the correct address. At least for now, who knows. OK? gcc/ChangeLog 2015-03-02 Ulrich Drepper * doc/invoke.texi (Opti

Re: [patch] PR rtl-optimization/65220: avoid integer division in stack alignment

2015-03-02 Thread Richard Henderson
On 02/26/2015 05:46 PM, Aldy Hernandez wrote: > +;; Optimize division or modulo by constant power of 2, if the constant > +;; materializes only after expansion. > +(define_insn_and_split "*udivmod4_pow2" > + [(set (match_operand:SWI48 0 "register_operand" "=r") > + (udiv:SWI48 (match_operand:S

Re: [PATCH] Fix PR65261

2015-03-02 Thread Marek Polacek
On Mon, Mar 02, 2015 at 06:22:49PM +0100, Markus Trippelsdorf wrote: > --- a/libcpp/lex.c > +++ b/libcpp/lex.c > @@ -519,6 +519,7 @@ init_vectorized_lexer (void) > and VSX unaligned loads (when VSX is available). This is otherwise > the same as the pre-GCC 5 version. */ > > +__attribute

[patch] libstdc++/65279 fix scoped_allocator_adaptor assignment

2015-03-02 Thread Jonathan Wakely
std::scoped_allocator_adaptor declares a move constructor but not a copy assignment operator, which means the copy assignment operator is defined as deleted, and so scoped_allocator_adaptor is not CopyAssignable. (This seems to be exactly what the standard requires, but that's clearly a defect in

Re: [PATCH] ubsan: improve bounds checking, add -fsanitize=bounds-strict

2015-03-02 Thread Martin Uecker
Marek Polacek : > On Fri, Feb 27, 2015 at 11:53:14AM -0800, Martin Uecker wrote: > > > > I tested Marek's proposed change and it works correctly, > > i.e. arrays which are not part of a struct are now > > instrumented when accessed through a pointer. This also > > means that the following case is

[PATCH] Fix PR65261

2015-03-02 Thread Markus Trippelsdorf
Running bootstrap-ubsan on ppc64le shows many instances of: libcpp/lex.c:552:30: runtime error: load of misaligned address 0x01001f31d37a for type 'const uchar', which requires 16 byte alignment But the unaligned vector loads are intended in this case, because they are preferable to forced-al

Re: [PATCH] Fix PR ipa/65087

2015-03-02 Thread Jan Hubicka
> Hello. > > Following one line patch is fix for PR ipa/65087. No regression has been > seen on x86_64-linux-pc. OK, it won't happend large programs, but can you arrange its execute method to return TODO_remove_functions only if some merging actually happened? Patch is preapproved with this chang

Re: [PATCH] Fix PR ipa/65263

2015-03-02 Thread Jan Hubicka
> > 2015-03-01 Martin Liska > Jan Hubicka > > PR ipa/65263 > * cgraph.c (cgraph_node::has_thunk_p): New function. > * cgraph.h (cgraph_node::has_thunk_p: Likewise. > * ipa-icf.c (redirect_all_callers): Do not redirect thunks. > (sem_function::merge):

[PATCH] Fix PR ipa/65087

2015-03-02 Thread Martin Liška
Hello. Following one line patch is fix for PR ipa/65087. No regression has been seen on x86_64-linux-pc. Ready for trunk? Thanks, Martin >From f39be56c66387c698546c498550d38512ddeb477 Mon Sep 17 00:00:00 2001 From: mliska Date: Mon, 2 Mar 2015 16:56:42 +0100 Subject: [PATCH] Fix PR ipa/65087.

Re: [patch] libstdc++/64367 fix __sso_string to compile with clang

2015-03-02 Thread Jonathan Wakely
The same file has another instance of the same problem (which compiles with G++ in all modes, but Clang rejects in c++98 mode). Tested x86_64-linux, committed to trunk. commit 7cf52d6430d73c412f5205098c6cfe0ce7a3d0d5 Author: Jonathan Wakely Date: Thu Feb 26 13:31:48 2015 + PR libstdc

Re: [wwwdocs, patch] Add libgccjit to gcc-5/changes.html

2015-03-02 Thread David Malcolm
On Fri, 2015-02-27 at 01:06 +0100, Gerald Pfeifer wrote: > Hi David, > > On Thu, 5 Feb 2015, David Malcolm wrote: > > This patch adds an -level section about libgccjit > > between the sections on languages and targets. > > this is a very nice idea. > > Index: htdocs/gcc-5/changes.html > ===

[PATCH] Fix PR ipa/65263

2015-03-02 Thread Martin Liška
Hello. This is suggested patch for the PR. Tested on x86_64-unknown-linux-pc. Ready for trunk? Thanks, Martin >From c0f86505df4519b555f110d8c9d320c17b9fcfd9 Mon Sep 17 00:00:00 2001 From: mliska Date: Mon, 2 Mar 2015 14:15:30 +0100 Subject: [PATCH] Fix PR ipa/65263. gcc/ChangeLog: 2015-03-01

Re: [PATCH] ubsan: improve bounds checking, add -fsanitize=bounds-strict

2015-03-02 Thread Marek Polacek
On Fri, Feb 27, 2015 at 11:53:14AM -0800, Martin Uecker wrote: > > I tested Marek's proposed change and it works correctly, > i.e. arrays which are not part of a struct are now > instrumented when accessed through a pointer. This also > means that the following case is diagnosed (correctly) > as u

Re: [PATCH] PR63175 - [4.9/5 regression] FAIL: gcc.dg/vect/costmodel/ppc/costmodel-bb-slp-9a.c scan-tree-dump-times slp2" basic block vectorized using SLP" 1

2015-03-02 Thread Martin Sebor
On 03/02/2015 06:58 AM, Richard Biener wrote: On Fri, 27 Feb 2015, Martin Sebor wrote: Given that Martin's fix to the testcase allowed it to succeed without Richi's fix for the underlying problem, is there a modification to the testcase or a new testcase that would really test the optimization?

Re: [PATCH, MPX runtime 1/2] Integrate MPX runtime library

2015-03-02 Thread Ilya Enkovich
On 09 Dec 23:56, Ilya Enkovich wrote: > 2014-12-09 22:57 GMT+03:00 Jeff Law : > > > > Wasn't this duplicated in the mpx-wrapper patch? > > Wrappers patch introduces similar option static-libmpxwrappers. > > > > > I think this is OK on the technical side. I need to do to some vote > > tallying on

RE: [PATCH][ARM]Automatically add -mthumb for thumb-only target when mode isn't specified

2015-03-02 Thread Kyrill Tkachov
> -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- > ow...@gcc.gnu.org] On Behalf Of James Greenhalgh > Sent: 02 March 2015 13:28 > To: Maxim Kuvyrkov > Cc: Terry Guo; gcc-patches@gcc.gnu.org; Ramana Radhakrishnan; Richard > Earnshaw > Subject: Re: [PATCH][ARM

Re: [patch, avr] Take 2: Fix PR64331: insn output and insn length computation rely on REG_DEAD notes.

2015-03-02 Thread Steven Bosscher
On Sat, Feb 28, 2015 at 5:38 PM, Georg-Johann Lay wrote: > Am 02/26/2015 um 11:45 PM schrieb Steven Bosscher: >> >> On Thu, Feb 26, 2015 at 8:35 PM, Georg-Johann Lay wrote: >>> >>> Take #2 introduces a new, avr-specific rtl pass whose sole purpose is to >>> rectify notes. The pass is scheduled rig

Re: [PATCH] PR63175 - [4.9/5 regression] FAIL: gcc.dg/vect/costmodel/ppc/costmodel-bb-slp-9a.c scan-tree-dump-times slp2" basic block vectorized using SLP" 1

2015-03-02 Thread Richard Biener
On Fri, 27 Feb 2015, Martin Sebor wrote: > > > Given that Martin's fix to the testcase allowed it to succeed without > > > Richi's fix for the underlying problem, is there a modification to the > > > testcase or a new testcase that would really test the optimization? > > > > Let me work on it. >

[PATCH] Fix PR65270

2015-03-02 Thread Richard Biener
This makes IPA ICF compare alignment of accesses. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2015-03-02 Richard Biener PR ipa/65270 * ipa-icf-gimple.c: Include builtins.h. (func_checker::compare_memory_operand): Compare base alignment. In

[PING][PATCH][4.9]Backport "Fix register corruption bug in ree"

2015-03-02 Thread Renlin Li
Ping~ Anybody has time to review it? Regards, Renlin Li On 06/02/15 17:48, Renlin Li wrote: Hi all, This is a backport patch for branch 4.9. You can find the original patch here: https://gcc.gnu.org/ml/gcc-patches/2014-09/msg00356.html And it has been commit on the trunk as r215205. This f

Re: [PATCH] PR target/65248: [5 Regression] Copy relocation in PIE against protected symbol

2015-03-02 Thread H.J. Lu
On Mon, Mar 2, 2015 at 5:39 AM, H.J. Lu wrote: > On Mon, Mar 2, 2015 at 4:05 AM, Richard Biener > wrote: >> On Mon, Mar 2, 2015 at 11:09 AM, Alan Modra wrote: >>> On Mon, Mar 02, 2015 at 09:40:01AM +0100, Richard Biener wrote: On Sat, Feb 28, 2015 at 5:42 PM, H.J. Lu wrote: > Ue copy

Re: [PATCH] PR target/65248: [5 Regression] Copy relocation in PIE against protected symbol

2015-03-02 Thread H.J. Lu
On Mon, Mar 2, 2015 at 4:05 AM, Richard Biener wrote: > On Mon, Mar 2, 2015 at 11:09 AM, Alan Modra wrote: >> On Mon, Mar 02, 2015 at 09:40:01AM +0100, Richard Biener wrote: >>> On Sat, Feb 28, 2015 at 5:42 PM, H.J. Lu wrote: >>> > Ue copy relocation in PIE improves performance. But copy reloca

Re: [PATCH] PR target/65248: [5 Regression] Copy relocation in PIE against protected symbol

2015-03-02 Thread H.J. Lu
On Sun, Mar 1, 2015 at 11:48 PM, Uros Bizjak wrote: > On Sat, Feb 28, 2015 at 5:42 PM, H.J. Lu wrote: >> Ue copy relocation in PIE improves performance. But copy relocation >> can't be used to access protected symbols defined in shared libaries >> and linker in binutils 2.26 enforces doesn't all

Re: [PATCH][ARM]Automatically add -mthumb for thumb-only target when mode isn't specified

2015-03-02 Thread James Greenhalgh
On Mon, Mar 02, 2015 at 01:08:13PM +, Maxim Kuvyrkov wrote: > > On Mar 2, 2015, at 4:44 AM, Terry Guo wrote: > > > > Hi there, > > > > If target mode isn't specified via either gcc configuration option > > --with-mode or command line, this patch intends to improve gcc driver to > > automatic

Re: [patch, avr] Take 2: Fix PR64331: insn output and insn length computation rely on REG_DEAD notes.

2015-03-02 Thread Denis Chertykov
2015-03-02 15:32 GMT+03:00 Georg-Johann Lay : > Am 02/28/2015 um 09:02 AM schrieb Denis Chertykov: > >> 2015-02-27 1:45 GMT+03:00 Steven Bosscher : >>> >>> On Thu, Feb 26, 2015 at 8:35 PM, Georg-Johann Lay wrote: Take #2 introduces a new, avr-specific rtl pass whose sole purpose is to >>>

Re: [PATCH][ARM]Automatically add -mthumb for thumb-only target when mode isn't specified

2015-03-02 Thread Maxim Kuvyrkov
> On Mar 2, 2015, at 4:44 AM, Terry Guo wrote: > > Hi there, > > If target mode isn't specified via either gcc configuration option > --with-mode or command line, this patch intends to improve gcc driver to > automatically add option -mthumb for thumb-only target. Tested with gcc > regression te

Re: [PATCH, CHKP, PR target/65183] Avoid wrong pass local data usage

2015-03-02 Thread Ilya Enkovich
2015-03-02 15:25 GMT+03:00 Maxim Kuvyrkov : >> On Mar 2, 2015, at 1:41 PM, Ilya Enkovich wrote: >> >> Committed to trunk. > > > Hi Ilya, > > Was this approved offline? If so, it is a good habit to note in the > "Committed to trunk" message who approved it. This way we have reviewer's > name on

Re: [patch, avr] Take 2: Fix PR64331: insn output and insn length computation rely on REG_DEAD notes.

2015-03-02 Thread Georg-Johann Lay
Am 02/28/2015 um 09:02 AM schrieb Denis Chertykov: 2015-02-27 1:45 GMT+03:00 Steven Bosscher : On Thu, Feb 26, 2015 at 8:35 PM, Georg-Johann Lay wrote: Take #2 introduces a new, avr-specific rtl pass whose sole purpose is to rectify notes. The pass is scheduled right before cfg does down (righ

Re: [PATCH, CHKP, PR target/65183] Avoid wrong pass local data usage

2015-03-02 Thread Maxim Kuvyrkov
> On Mar 2, 2015, at 1:41 PM, Ilya Enkovich wrote: > > Committed to trunk. Hi Ilya, Was this approved offline? If so, it is a good habit to note in the "Committed to trunk" message who approved it. This way we have reviewer's name on file. Thank you, -- Maxim Kuvyrkov www.linaro.org >

Re: [PATCH] PR target/65248: [5 Regression] Copy relocation in PIE against protected symbol

2015-03-02 Thread Richard Biener
On Mon, Mar 2, 2015 at 11:09 AM, Alan Modra wrote: > On Mon, Mar 02, 2015 at 09:40:01AM +0100, Richard Biener wrote: >> On Sat, Feb 28, 2015 at 5:42 PM, H.J. Lu wrote: >> > Ue copy relocation in PIE improves performance. But copy relocation >> > can't be used to access protected symbols defined

[Ada] Compiler hang with full inlining and use clause in parent private part

2015-03-02 Thread Arnaud Charlet
When compiling a descendant instance unit use_clauses in parent units are chained to simplify removal at end of compilation. Use_clauses that appear in the private part of parent are chained when compiling the private part of the descendant. To prevent circularities in the list, use_clauses in the

[Ada] Type_Invariant'Class on interface types

2015-03-02 Thread Arnaud Charlet
This new language feature allows Type_Invariant'class on interface types. When a type implements one or several interfaces, its inherited type invariant is the conjunction of all ancestor Type_Invariant'Class. The following package now compiles without errors: package AI12_0041 is type VIntf is

[Ada] Argument of pragma Depends has wrong source location

2015-03-02 Thread Arnaud Charlet
This patch ensures that the normalization mechanism which services aspects or pragmas Abstract_State, Contract_State, Depends, Global, Initializes, Refined_Depends, Refined_Global and Refined_State properly sets the source location of the first item. No change in behavior. No reproducer possible as

[Ada] Initial checkin for unnesting of subprograms

2015-03-02 Thread Arnaud Charlet
This is an initial checkin for the feature of unnesting subprograms in the front end (for use by alternate, non GCC backends). The documentation is complete, but the implementation checked in is only part of the final implementation, so no test needed yet. Tested on x86_64-pc-linux-gnu, committed

Re: [PATCH, CHKP, i386, PR target/65184] Fix pass_by_reference for MS ABI for bounds

2015-03-02 Thread Ilya Enkovich
Committed to trunk. 2015-02-25 17:25 GMT+03:00 Ilya Enkovich : > Hi, > > Currenly ix86_pass_by_reference may return 1 for bounds if MS ABI is used. > This patch explicitly says bounds are never passed by reference. > Bootstrapped and tested on x86_64-unknown-linux-gnu. OK for trunk? > > > Tha

Re: [PATCH, CHKP, PR target/65183] Avoid wrong pass local data usage

2015-03-02 Thread Ilya Enkovich
Committed to trunk. 2015-02-25 12:19 GMT+03:00 Ilya Enkovich : > Hi, > > This patch fixes a case when outdated checker local data is used to process > external calls. Bootstrapped and tested on x86_64-unknown-linux-gnu. OK for > trunk? > > Thanks, > Ilya > -- > gcc/ > > 2015-02-25 Ilya Enkovi

Re: [PATCH, CHKP, PR target/65044] Restrict pointer bounds checker with Sanitizer

2015-03-02 Thread Ilya Enkovich
Ping 2015-02-16 20:15 GMT+03:00 Ilya Enkovich : > On 16 Feb 17:01, Jakub Jelinek wrote: >> On Mon, Feb 16, 2015 at 06:56:45PM +0300, Ilya Enkovich wrote: >> > On 16 Feb 16:31, Jakub Jelinek wrote: >> > > On Mon, Feb 16, 2015 at 06:20:59PM +0300, Ilya Enkovich wrote: >> > > > This patch restricts u

[committed] add new roles to MAINTAINERS

2015-03-02 Thread Ilya Enkovich
Hi, I've checked in this patch adding new roles for myself. Thanks, Ilya -- 2015-03-02 Ilya Enkovich * MAINTAINERS (Various Maintainers): Add myself as libmpx, pointer bounds checker and i386 MPX maintainer. Index: MAINTAINERS ===

Re: [PATCH] PR target/65248: [5 Regression] Copy relocation in PIE against protected symbol

2015-03-02 Thread Alan Modra
On Mon, Mar 02, 2015 at 09:40:01AM +0100, Richard Biener wrote: > On Sat, Feb 28, 2015 at 5:42 PM, H.J. Lu wrote: > > Ue copy relocation in PIE improves performance. But copy relocation > > can't be used to access protected symbols defined in shared libaries > > and linker in binutils 2.26 enforc

Re: [PATCH] ICF: move readonly decision for variables to the right place

2015-03-02 Thread Martin Liška
On 03/02/2015 08:53 AM, Jan Hubicka wrote: Hi, this is a variant of patch I commited. It takes care to load the constructor to memory in sem_variable::equals and donot touch it earlier. I also made sem_variable::parse to skip volatile and reigster variables. Bootstrapped/regtested x86_64-linux,

Re: [Patch, Fortran] Fix ATOMIC checks

2015-03-02 Thread Paul Richard Thomas
Dear Tobias, This looks kind of 'obvious' to me! OK for trunk. Thanks for the patch. Paul On 1 March 2015 at 23:35, Tobias Burnus wrote: > This patch changes the coarray/coindexed check to match what the error > message already claimed. > > OK for the trunk? > > Tobias -- Outside of a dog,

[Ada] Output of errors related to Pre'Class, Pre_Class, Post'Class, Post_Class

2015-03-02 Thread Arnaud Charlet
This patch takes advantage of the recently introduced mechanism that outputs special names _Pre and _Post to emit errors related to pre/postconditions. No change in behavior, no test. Tested on x86_64-pc-linux-gnu, committed on trunk 2015-03-02 Hristian Kirtchev * sem_prag.adb (Duplic

Re: [PATCH] PR target/65248: [5 Regression] Copy relocation in PIE against protected symbol

2015-03-02 Thread Richard Biener
On Sat, Feb 28, 2015 at 5:42 PM, H.J. Lu wrote: > Ue copy relocation in PIE improves performance. But copy relocation > can't be used to access protected symbols defined in shared libaries > and linker in binutils 2.26 enforces doesn't allow it. GCC doesn't > know if an external definition is pr

Re: [patch/committed] PR middle-end/65233 make walk-ssa_copies handle empty PHIs

2015-03-02 Thread Richard Biener
On Mon, Mar 2, 2015 at 6:34 AM, Aldy Hernandez wrote: > As I mention in the PR... > > What's happening here is that the ipa_polymorphic_call_context constructor > is calling walk_ssa_copies on a PHI node that has no arguments. This > happens because finalize_jump_threads eventually removes some P