[PATCH, nds32] Committed: Look into LO_SUM's operand to determine whether it is a valid address.

2014-11-20 Thread Chung-Ju Wu
Hi, all, In the nds32_legitimate_address_p function, we should look into LO_SUM's operand so that we can precisely identify if it is a valid address. This can help to avoid extra move instruction and improve code generation. gcc/ChangeLog 2014-11-21 Chung-Ju Wu * config/nds32/nds32.

Re: [PATCH 1/2] PR debug/38757 gcc does not emit DW_LANG_C99.

2014-11-20 Thread Jakub Jelinek
On Thu, Nov 20, 2014 at 11:30:11PM +0100, Mark Wielaard wrote: > --- a/gcc/config/avr/avr-c.c > +++ b/gcc/config/avr/avr-c.c > @@ -386,7 +386,8 @@ avr_cpu_cpp_builtins (struct cpp_reader *pfile) > (as mentioned in ISO/IEC DTR 18037; Annex F.2) which is not > implemented in GCC up to now

[PATCH, nds32] Committed: Fix CFI info for variadic function arguments.

2014-11-20 Thread Chung-Ju Wu
Hi, all, In nds32 ABI, the vararg arguments are pushed onto stack by callee. However, those registers content are unnecessary to be restored. We need to create an extra register note to inform gcc such information so that it can generate correct CFI info for variadic function. gcc/ChangeLog 201

Re: [PATCH] Fix tree-ssa-strlen ICE introduced by r211956 (PR tree-optimization/61773)

2014-11-20 Thread Richard Biener
On November 20, 2014 7:16:04 PM CET, Jakub Jelinek wrote: >Hi! > >Before the r211956 changes, the only places that set si->stmt >were required to check that stpcpy has been declared (with the right >prototype) to signal the strlen pass that it can use stpcpy for >optimization. But r211956 sets si

[PATCH, nds32] Committed: Add missing RTX_FRAME_RELATED_P to set fp_adjust_insn rtx.

2014-11-20 Thread Chung-Ju Wu
Hi, all, There is a wrong CFI info when frame pointer is needed. The root cause is that we forgot to set fp_adjust_insn as RTX_FRAME_RELATED_P rtx. Fix this issue with following patch. gcc/ChangeLog 2014-11-21 Chung-Ju Wu * config/nds32/nds32.c (nds32_expand_prologue): Set fp_adjus

Re: [PATCH, nds32] Committed: Complete -march= option design.

2014-11-20 Thread Chung-Ju Wu
2014-11-21 14:33 GMT+08:00 Chung-Ju Wu : > Hi, all, [snip] > > gcc/ChangeLog > > 2014-11-21 Chung-Ju Wu > > * config/nds32/nds32.opt (march): Add help message. > Sorry for that I forgot to attach patch file in the previous mail. Best regards, jasonwucj 0001-Complete-march-option-desi

[PATCH, nds32] Committed: Complete -march= option design.

2014-11-20 Thread Chung-Ju Wu
Hi, all, When the help option --target-help is issued, the available items for -march=X are supposed to be displayed automatically. We need to complete -march= option design to perform such mechanism. gcc/ChangeLog 2014-11-21 Chung-Ju Wu * config/nds32/nds32.opt (march): Add help me

[PATCH, combine] Try REG_EQUAL for nonzero_bits

2014-11-20 Thread Zhenqiang Chen
Hi, The patch tries to use REG_EQUAL to get more precise info for nonzero_bits, which helps to remove unnecessary zero_extend. Here is an example when compiling Coremark, we have rtx like, (insn 1244 386 388 47 (set (reg:SI 263 [ D.5767 ]) (reg:SI 384 [ D.5767 ])) 786 {*thumb2_movsi_insn

[PATCH] Fix PR 63952 (Re: [PATCH, ifcvt] Allow CC mode if HAVE_cbranchcc4)

2014-11-20 Thread Ulrich Weigand
Richard Biener wrote: > This probably caused bootstrap on s390x-linux to fail as in PR63952 > (last checked with rev. 217714). It seems we have both a back-end bug and a middle-end bug here. First of all, this code in optabs.c:prepare_cmp_insn is quite strange: if (GET_MODE_CLASS (mode) == M

C++ PATCH for c++/63658 (reference non-type template parameters)

2014-11-20 Thread Jason Merrill
My patch for c++/58606 changed the internal representation of reference template arguments to include the dereference in some places, but not enough. Tested x86_64-cp-linux-gnu, applying to trunk and 4.9. commit f7ea463fa794117000ef8495cb87d8c054323c7e Author: Jason Merrill Date: Thu Nov 20

Re: PATCH, PR 63965, rs6000, add support for scalar floating point in Altivec registers

2014-11-20 Thread David Edelsohn
On Thu, Nov 20, 2014 at 4:48 PM, Michael Meissner wrote: > This patch fixes the failures in the following tests if --with-cpu is not used > to set the cpu to cpus with vsx enabled (power7/power8): > > gcc.target/powerpc/altivec-splat.c > gcc.target/powerpc/pr27158.c > > I have buil

Re: [PATCH, rs6000] Fix PR63673 -- missing prototype for vec_vsx_ld/vec_vsx_st

2014-11-20 Thread David Edelsohn
On Thu, Nov 20, 2014 at 6:49 PM, Bill Schmidt wrote: > Hi, > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63673 observes that > vec_vsx_ld doesn't currently permit the base pointer to have a type of > pointer to double, even though this is specified as legal in the GCC > user's manual. We do al

Re: [PATCH] rs6000: Follow up for signed integer overflow fix

2014-11-20 Thread David Edelsohn
On Thu, Nov 20, 2014 at 1:58 PM, Markus Trippelsdorf wrote: > On 2014.11.20 at 19:44 +0100, Jakub Jelinek wrote: >> On Thu, Nov 20, 2014 at 07:41:43PM +0100, Markus Trippelsdorf wrote: >> > 2014-11-20 Markus Trippelsdorf >> > >> > * config/rs6000/rs6000.c (includes_rldic_lshift_p): Cast 0 t

Re: [PATCH] OpenACC for C front end

2014-11-20 Thread James Norris
Hi! On 11/13/2014 09:04 AM, Jakub Jelinek wrote: On Wed, Nov 05, 2014 at 03:39:44PM -0600, James Norris wrote: * c-typeck.c (c_finish_oacc_parallel, c_finish_oacc_kernels, c_finish_oacc_data): New functions. (handle_omp_array_sections, c_finish_omp_clauses): Handle shou

[PATCH, rs6000] Fix PR63673 -- missing prototype for vec_vsx_ld/vec_vsx_st

2014-11-20 Thread Bill Schmidt
Hi, https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63673 observes that vec_vsx_ld doesn't currently permit the base pointer to have a type of pointer to double, even though this is specified as legal in the GCC user's manual. We do allow a pointer to float, so this is just an oversight. This patch

Re: [PATCH] gcc/c-family/c-cppbuiltin.c: Use 20 instead of 18 for the maximized 64-bits integer decimal string length

2014-11-20 Thread Chen Gang
OK, thank you for your details description. I shall send patch v2 for it, within this week. Send from Lenovo A788t. Joseph Myers wrote: >On Thu, 20 Nov 2014, Chen Gang wrote: > >> OK, thanks. I guess your meaning is: >> >> - If the value is small enough to be expressed by type 'int', we sho

Re: [PATCH] OpenACC for C++ front end

2014-11-20 Thread James Norris
Hi! On 11/13/2014 07:02 AM, Jakub Jelinek wrote: On Wed, Nov 05, 2014 at 03:37:08PM -0600, James Norris wrote: 2014-11-05 James Norris Cesar Philippidis Thomas Schwinge Ilmir Usmanov ... Please check formatting. I see various spots with 8 spaces i

Re: [PATCH] gcc/c-family/c-cppbuiltin.c: Use 20 instead of 18 for the maximized 64-bits integer decimal string length

2014-11-20 Thread Joseph Myers
On Thu, 20 Nov 2014, Chen Gang wrote: > OK, thanks. I guess your meaning is: > > - If the value is small enough to be expressed by type 'int', we should >not provide 'LL'. Yes - and values not small enough should probably not be accepted by this function. > - If the value is positive num

[PATCH 2/2] PR debug/38757 continued. Handle C11, C++11 and C++14.

2014-11-20 Thread Mark Wielaard
This change depends on the new DWARFv5 constants mentioned in the following draft: http://dwarfstd.org/doc/dwarf5.20141029.pdf gcc/ChangeLog * dwarf2out.c (is_cxx): Add DW_LANG_C_plus_plus_11 and DW_LANG_C_plus_plus_14. (lower_bound_default): Likewise. Plus DW_LANG_C11.

[PATCH 1/2] PR debug/38757 gcc does not emit DW_LANG_C99.

2014-11-20 Thread Mark Wielaard
On Thu, Nov 13, 2014 at 02:19:57PM +0100, Richard Biener wrote: > I'm fine with the string solution as well. Just properly transfer whatever > info you want through the translation unit decl and LTO stream it. OK, strings it is then. Here are two patches. The first makes the change to the lang_ho

Re: [PATCH 1/2] teach mklog to get name / email from git config when available

2014-11-20 Thread Tom de Vries
On 20-11-14 17:43, Segher Boessenkool wrote: On Thu, Nov 20, 2014 at 05:22:20PM +0100, Tom de Vries wrote: +my $conf = "$ENV{HOME}/.mklog"; +if (-f "$conf") { +open (CONF, "$conf") + or die "Could not open file '$conf' for reading: $!\n"; +while () { + if (m/^\s*NAME\s*=\s*(.

Re: PATCH, PR 63965, rs6000, add support for scalar floating point in Altivec registers

2014-11-20 Thread Michael Meissner
This patch fixes the failures in the following tests if --with-cpu is not used to set the cpu to cpus with vsx enabled (power7/power8): gcc.target/powerpc/altivec-splat.c gcc.target/powerpc/pr27158.c I have built these patches on power7 systems (default cpu = power7, and power6),

Re: [PATCH] PR lto/63968: 175.vpr from cpu2000 fails to build with LTO

2014-11-20 Thread Jan Hubicka
> Hello. > > As I reimplemented fibheap to C++ template, Honza told me that replace_key > method actually > supports just decrement operation. Old implementation suppress any feedback > if we try to increase key: > > fibheap.c: > ... > /* If we wanted to, we could actually do a real increase

Re: LTO streaming of TARGET_OPTIMIZE_NODE

2014-11-20 Thread Jan Hubicka
> On 11/20/2014 02:20 PM, Richard Biener wrote: > >On Thu, 20 Nov 2014, Bernd Schmidt wrote: > > > >>On 11/13/2014 05:06 AM, Jan Hubicka wrote: > >>>this patch adds infrastructure for proper streaming and merging of > >>>TREE_TARGET_OPTION. > >> > >>This breaks the offloading path via LTO since it

Re: [patch] Warn on undefined loop exit

2014-11-20 Thread Andrew Stubbs
On 20/11/14 16:27, Richard Biener wrote: If it's really duplicated code can you split it out to a function? + if (OPT_Waggressive_loop_optimizations) +{ this doesn't do what you think it does ;) The variable to check is warn_aggressive_loop_optimizations. + if (exit_warned && pr

Re: [patch] Warn on undefined loop exit

2014-11-20 Thread Marek Polacek
On Thu, Nov 20, 2014 at 05:27:35PM +0100, Richard Biener wrote: > + if (exit_warned && problem_stmts != vNULL) > +{ > > !problem_stmts.empty () /home/marek/src/gcc/gcc/tree-ssa-loop-niter.c: In function ‘void maybe_lower_iteration_bound(loop*)’: /home/marek/src/gcc/gcc/tree-ssa-loop

Re: [patch] Flatten streamer header files Pt. 1

2014-11-20 Thread Andrew MacLeod
On 11/20/2014 03:05 PM, Michael Collison wrote: This is a part one of two part patch that flattens gimple-streamer.h, lto-streamer.h and tree-streamer.h. This work is part of the GCC Re-Architecture effort being led by Andrew MacLeod. In gimple-streamer.h I moved all exports for gimple-streame

[patch] Flatten streamer header files Pt. 2

2014-11-20 Thread Michael Collison
This is a final part of two part patch that flattens gimple-streamer.h, lto-streamer.h and tree-streamer.h. This patch modifies all source files that previously included the three streamer header files to now include the header file(s) that contains the exports that it needs. I performed

[patch] Flatten streamer header files Pt. 1

2014-11-20 Thread Michael Collison
This is a part one of two part patch that flattens gimple-streamer.h, lto-streamer.h and tree-streamer.h. This work is part of the GCC Re-Architecture effort being led by Andrew MacLeod. In gimple-streamer.h I moved all exports for gimple-streamer-in.c to a new file gimple-streamer-in.h. I als

[patch c++]: Fix PR/53904

2014-11-20 Thread Kai Tietz
Hello, this issue fixes a type-overflow issue caused by trying to cast a UHWI via tree_to_shwi. As soon as value gets larger then SHWI_MAX, we get an error for it. So we need to cast it via tree_to_uhwi, and then casting it to the signed variant. ChangeLog 2014-11-20 Kai Tietz PR c++/639

Re: [PATCH] Fix ubsan and C++14 constexpr ICEs (PR sanitizer/63956)

2014-11-20 Thread Marek Polacek
On Thu, Nov 20, 2014 at 06:27:25PM +0100, Jakub Jelinek wrote: > On Thu, Nov 20, 2014 at 06:14:52PM +0100, Marek Polacek wrote: > > + if (!current_function_decl && is_ubsan_builtin_p (fun)) > > +return void_node; > > + > > I don't understand the !current_function_decl here. That is because

Re: [PATCH] Fix ICEs in simplify_immed_subreg on OImode/XImode subregs (PR target/63910)

2014-11-20 Thread Richard Sandiford
Jakub Jelinek writes: > On Wed, Nov 19, 2014 at 02:23:47PM -0800, Mike Stump wrote: >> On Nov 19, 2014, at 1:57 PM, Jakub Jelinek wrote: >> > Though, following patch is just fine for me too, I don't think it will >> > make a significant difference: >> >> This version is fine by me. > > Richard,

Re: [PATCH] rs6000: Follow up for signed integer overflow fix

2014-11-20 Thread Markus Trippelsdorf
On 2014.11.20 at 19:44 +0100, Jakub Jelinek wrote: > On Thu, Nov 20, 2014 at 07:41:43PM +0100, Markus Trippelsdorf wrote: > > 2014-11-20 Markus Trippelsdorf > > > > * config/rs6000/rs6000.c (includes_rldic_lshift_p): Cast 0 to unsigned. > > (includes_rldicr_lshift_p): Likewise. > > > >

Re: [PATCH] Fix ICE with non-lvalue vector subscripts and make sure non-lvalue vector subscripts aren't used as lvalues (PR target/63764)

2014-11-20 Thread Richard Henderson
On 11/20/2014 07:09 PM, Jakub Jelinek wrote: > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? > > 2014-11-20 Jakub Jelinek > > PR target/63764 > c-family/ > * c-common.h (convert_vector_to_pointer_for_subscript): Change > return type to bool. > * c

Re: [PATCH] rs6000: Follow up for signed integer overflow fix

2014-11-20 Thread Jakub Jelinek
On Thu, Nov 20, 2014 at 07:41:43PM +0100, Markus Trippelsdorf wrote: > 2014-11-20 Markus Trippelsdorf > > * config/rs6000/rs6000.c (includes_rldic_lshift_p): Cast 0 to unsigned. > (includes_rldicr_lshift_p): Likewise. > > diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/

[PATCH] rs6000: Follow up for signed integer overflow fix

2014-11-20 Thread Markus Trippelsdorf
On 2014.11.20 at 08:59 -0500, David Edelsohn wrote: > On Thu, Nov 20, 2014 at 8:27 AM, Markus Trippelsdorf > wrote: > > Running the testsuite after bootstrap-ubsan on gcc112 shows several issues. > > See > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63426 for the full list. > > > > This patch

Re: [PATCH, ifcvt] Fix PR63917

2014-11-20 Thread Richard Henderson
On 11/20/2014 10:48 AM, Zhenqiang Chen wrote: > +/* Check X clobber CC reg or not. */ > + > +static bool > +clobber_cc_p (rtx x) > +{ > + RTX_CODE code = GET_CODE (x); > + int i; > + > + if (code == CLOBBER > + && REG_P (XEXP (x, 0)) > + && (GET_MODE_CLASS (GET_MODE (XEXP (x, 0))) ==

[ia64 PATCH] Fix up ia64 attribute handling (PR target/61137)

2014-11-20 Thread Jakub Jelinek
Hi! Seems the gcc.target/ia64/small-addr-1.c testcase is failing on ia64 since r210262 but clearly has been failing for much longer if compiled with C++ (just there is insufficient testsuite coverage). The problem is that for the model attribute (and apparently common_object on VMS too), the argum

Re: New patch: [AArch64] [BE] [1/2] Make large opaque integer modes endianness-safe.

2014-11-20 Thread Marcus Shawcroft
On 13 November 2014 10:09, David Sherwood wrote: > gcc/: > 2014-11-13 David Sherwood > > * config/aarch64/aarch64-protos.h (aarch64_simd_attr_length_rglist, > aarch64_reverse_mask): New decls. > * config/aarch64/iterators.md (UNSPEC_REV_REGLIST): New enum. >

[PATCH] Fix tree-ssa-strlen ICE introduced by r211956 (PR tree-optimization/61773)

2014-11-20 Thread Jakub Jelinek
Hi! Before the r211956 changes, the only places that set si->stmt were required to check that stpcpy has been declared (with the right prototype) to signal the strlen pass that it can use stpcpy for optimization. But r211956 sets si->stmt also for malloca call, which isn't in any way related to s

Re: [Aarch64][BE][2/2] Fix vector load/stores to not use ld1/st1

2014-11-20 Thread Marcus Shawcroft
On 14 November 2014 16:48, Alan Hayward wrote: > This is a new version of my BE patch from a few weeks ago. > This is part 2 and covers all the aarch64 changes. > > When combined with the first patch, It fixes up movoi/ci/xi for Big > Endian, so that we end up with the lab of a big-endian integer

Re: SRA: don't drop clobbers

2014-11-20 Thread Martin Jambor
Hi, On Mon, Nov 03, 2014 at 10:46:49PM +0100, Marc Glisse wrote: > On Mon, 3 Nov 2014, Marc Glisse wrote: > > >On Mon, 3 Nov 2014, Martin Jambor wrote: > > > >>I just applied your patch on top of trunk revision 217032 on my > > > >Ah, that explains it, thanks. This patch is a follow-up to > >r217

[PATCH] Fix ICE with non-lvalue vector subscripts and make sure non-lvalue vector subscripts aren't used as lvalues (PR target/63764)

2014-11-20 Thread Jakub Jelinek
Hi! This patch fixes ICEs if a non-lvalue vector (say cast of one vector to another vector type) was subscripted and used as lhs. The following patch, if *vecp is not lvalue, will copy it to a temporary variable which can be made addressable for the subscription, and afterwards wrap it into a NON_

Re: [PATCH] Fix ICEs in simplify_immed_subreg on OImode/XImode subregs (PR target/63910)

2014-11-20 Thread Jakub Jelinek
On Wed, Nov 19, 2014 at 02:23:47PM -0800, Mike Stump wrote: > On Nov 19, 2014, at 1:57 PM, Jakub Jelinek wrote: > > Though, following patch is just fine for me too, I don't think it will > > make a significant difference: > > This version is fine by me. Richard, are you ok with that too? Bootst

[gomp4] fix a fortran bootstrap failure

2014-11-20 Thread Cesar Philippidis
This patch resolves a bootstrap failure in gomp-4_0-branch, which I probably introduced after I switched the cache error message from a gfc_error to a sorry. The code parameter isn't being used anymore by resolve_oacc_cache, so I've explicitly marked it as unused. I've applied this patch to gomp-4

Re: [PATCH] Fix ubsan and C++14 constexpr ICEs (PR sanitizer/63956)

2014-11-20 Thread Jakub Jelinek
On Thu, Nov 20, 2014 at 06:14:52PM +0100, Marek Polacek wrote: > This patch fixes a bunch of ICEs related to C++14 constexprs and > -fsanitize=undefined. We should ignore ubsan internal functions > and ubsan builtins in constexpr functions in cxx_eval_call_expression. > > Also add proper printing

[PATCH] Fix ubsan and C++14 constexpr ICEs (PR sanitizer/63956)

2014-11-20 Thread Marek Polacek
This patch fixes a bunch of ICEs related to C++14 constexprs and -fsanitize=undefined. We should ignore ubsan internal functions and ubsan builtins in constexpr functions in cxx_eval_call_expression. Also add proper printing of internal functions into the C++ printer. Bootstrapped/regtested on p

Re: [PATCH][ARM] Make issue rate part of per-core tuning structs

2014-11-20 Thread Kyrill Tkachov
I should say that the patch context depends on the macro fusion hook implementation posted here: https://gcc.gnu.org/ml/gcc-patches/2014-11/msg00958.html Kyrill On 20/11/14 16:43, Kyrill Tkachov wrote: Hi all, This patch makes the arm_issue_rate function lookup the issue rate of the process f

Re: [AArch64, Patch] Add range-check for Symbol + offset addressing.

2014-11-20 Thread Marcus Shawcroft
On 20 November 2014 14:33, Tejas Belagod wrote: > The same patch applies cleanly to 4.9. OK to commit? > > Thanks, > Tejas. Provided it regresses ok, yes. /Marcus

Re: [PATCH 1/2] teach mklog to get name / email from git config when available

2014-11-20 Thread Segher Boessenkool
On Thu, Nov 20, 2014 at 05:22:20PM +0100, Tom de Vries wrote: > +my $conf = "$ENV{HOME}/.mklog"; > +if (-f "$conf") { > +open (CONF, "$conf") > + or die "Could not open file '$conf' for reading: $!\n"; > +while () { > + if (m/^\s*NAME\s*=\s*(.*)\s*$/) { The final \s* never matches

[PATCH][ARM] Make issue rate part of per-core tuning structs

2014-11-20 Thread Kyrill Tkachov
Hi all, This patch makes the arm_issue_rate function lookup the issue rate of the process from the tuning structs. This makes it look more like the aarch64 mechanism and centralises a processor-specific construct to the tuning structs, thus not forcing us to remember to update the arm_issue_ra

Re: [PATCH, i386] Add new arg values for __builtin_cpu_supports

2014-11-20 Thread Jakub Jelinek
On Thu, Nov 20, 2014 at 07:36:03PM +0300, Ilya Enkovich wrote: > Hi, > > MPX runtime checks some feature bits in order to check MPX is fully > supported. Runtime does it by cpuid calls but there is a > __builtin_cpu_supports which may be used for that. Unfortunately > currently it doesn't suppor

Re: [PATCH] PR ipa/63909 ICE: SIGSEGV in ipa_icf_gimple::func_checker::compare_bb()

2014-11-20 Thread Richard Biener
On Thu, Nov 20, 2014 at 5:30 PM, Martin Liška wrote: > Hello. > > Following patch fixes ICE in IPA ICF. Problem was that number of non-debug > statements in a BB can > change (for instance by IPA split), so that the number is recomputed. Huh, so can it get different for both candidates? I think

[PATCH, i386] Add new arg values for __builtin_cpu_supports

2014-11-20 Thread Ilya Enkovich
Hi, MPX runtime checks some feature bits in order to check MPX is fully supported. Runtime does it by cpuid calls but there is a __builtin_cpu_supports which may be used for that. Unfortunately currently it doesn't support required bits. Will it be OK to add them for trunk? Thanks, Ilya --

[PATCH] PR ipa/63909 ICE: SIGSEGV in ipa_icf_gimple::func_checker::compare_bb()

2014-11-20 Thread Martin Liška
Hello. Following patch fixes ICE in IPA ICF. Problem was that number of non-debug statements in a BB can change (for instance by IPA split), so that the number is recomputed. Patch can bootstrap on x86_64-linux-pc and no regression has been seen. Ready for trunk? Thanks, Martin gcc/ChangeLog:

Re: [PATCH][AArch64] Add bounds checking to vqdm*_lane intrinsics via a qualifier that also flips endianness

2014-11-20 Thread Charles Baylis
On 20 November 2014 07:49, Marcus Shawcroft wrote: > On 19 November 2014 19:05, Charles Baylis wrote: > >> PR target/63870 >> * config/aarch64/aarch64-builtins.c (aarch64_simd_expand_args): Pass >> expression to aarch64_simd_lane_bounds. >> * config/aarch64/aarch64

Add to maintainers list.

2014-11-20 Thread Alex Velenko
2014-11-20 Alex Velenko *MAINTAINERS (write-after-approval): Add myself. diff --git a/MAINTAINERS b/MAINTAINERS index 11a28ef..eada4e9 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -566,6 +566,7 @@ David Ung Neil Vachharajani

Re: [patch] Warn on undefined loop exit

2014-11-20 Thread Richard Biener
On Wed, Nov 19, 2014 at 9:19 PM, Andrew Stubbs wrote: > On 19/11/14 16:39, Marek Polacek wrote: >> >> On Wed, Nov 19, 2014 at 04:32:43PM +, Andrew Stubbs wrote: >>> >>> +if (warning_at (gimple_location (elt->stmt), >>> +OPT_Waggressive_lo

Re: [PATCH x86, PR60451] Expand even/odd permutation using pack insn.

2014-11-20 Thread Evgeny Stupachenko
Bootstrap / make check passed with updated patch. Is it still ok? It looks like we don't need "expand_vec_perm_vpshufb2_vpermq_even_odd" any more with the patch. However the clean up will be in the separate patch after appropriate testing. Modified ChangeLog: 2014-11-20 Evgeny Stupachenko g

Re: [PATCH 1/2] teach mklog to get name / email from git config when available

2014-11-20 Thread Tom de Vries
On 09-05-14 16:47, Diego Novillo wrote: I would probably use git config directly here. It would work with both git and svn checkouts (if you have a global .git configuration). But testing for .git is fine with me as well. I like Peter's idea of having a ~/.mklog file to override. This would work

Re: [PATCH 10/21] PR jit/63854: Fix leak of worklist within jit-recording.c

2014-11-20 Thread Jeff Law
On 11/20/14 09:01, Richard Biener wrote: Is there a governance distinction here, between patch review vs decisions of the steering committee? i.e. do changes to the maintainers part of the MAINTAINERS file require higher-level approval? Yes, reviewers and maintainers are appointed by the steer

[Ada] PR ada/63931

2014-11-20 Thread Arnaud Charlet
Fixing version number according to new GCC naming scheme. PR ada/63931 * gnatvsn.ads (Library_Version): Switch to "5". Index: gnatvsn.ads === --- gnatvsn.ads (revision 217874) +++ gn

[PR63762][4.9] Backport the patch which fixes "GCC generates UNPREDICTABLE STR with Rn = Rt for arm"

2014-11-20 Thread Renlin Li
Hi all, This is a backport for gcc-4_9-branch of the patch "[PR63762]GCC generates UNPREDICTABLE STR with Rn = Rt for arm" posted in: https://gcc.gnu.org/ml/gcc-patches/2014-11/msg02253.html arm-none-eabi has been test on the model, no new issues. bootstrapping and regression tested on x86,

Re: "gimple-classes-v2-option-3" git branch committed to svn trunk as r217787

2014-11-20 Thread Richard Biener
On Thu, Nov 20, 2014 at 4:34 PM, Michael Matz wrote: > Hi, > > On Thu, 20 Nov 2014, Richard Biener wrote: > >> > I don't think this API will make the non-C++-fans happier; I think the >> > objection to the work I just merged is that it's adding more C++ than >> > those people are comfortable with.

Re: [PATCH 10/21] PR jit/63854: Fix leak of worklist within jit-recording.c

2014-11-20 Thread Richard Biener
On Wed, Nov 19, 2014 at 9:02 PM, David Malcolm wrote: > On Wed, 2014-11-19 at 09:59 -0700, Jeff Law wrote: >> On 11/19/14 03:46, David Malcolm wrote: >> > Fix this leak: >> > >> > 160 bytes in 5 blocks are definitely lost in loss record 154 of 228 >> > at 0x4A0645D: malloc (in >> > /usr/lib64

Re: [PATCH][doc] Document cortex-a17 and cortex-a17.cortex-a7 -m{cpu,tune} options

2014-11-20 Thread Richard Earnshaw
On 20/11/14 15:31, Kyrill Tkachov wrote: > Hi all, > > As Joseph reminded, new -mcpu options should be documented in > invoke.texi. This adds the documentation for the cortex-a17 and > cortex-a17.cortex-a7 values. > Ok to go in if the corresponding support patches posted earlier are > accepted?

Re: [PATCH] PR63426 Fix various signed integer overflows

2014-11-20 Thread Jakub Jelinek
On Thu, Nov 20, 2014 at 02:27:52PM +0100, Markus Trippelsdorf wrote: > 2014-11-20 Markus Trippelsdorf > > * emit-rtl.c (const_wide_int_htab_hash): Likewise. > * loop-iv.c (determine_max_iter): Likewise. > (iv_number_of_iterations): Likewise. > * tree-ssa-loop-ivopts.c (g

Re: [PATCH, committed] Update Automake files

2014-11-20 Thread Jan-Benedict Glaw
Hi Bernd, On Thu, 2014-11-20 14:34:00 +0100, Bernd Edlinger wrote: > > This patch updates the files taken from Automake. Committed. > > the updated version of "missing" will confuse the gmp-4.3.2 configure > script if it is installed in-tree with "contrib/download_prerequisites" > and flex is n

[Ada] Source in multi-unit source has unique object file name

2014-11-20 Thread Arnaud Charlet
Two units, one in a multi-source file and one in another source with the same base file name do not have the same object file name. No error during processing of the following project file should be reported: project Prj is package Naming is for Spec ("foo_bar") use "foo_bar.ads" at 2;

Re: [PATCH] sh: char isn't signed

2014-11-20 Thread Oleg Endo
On Thu, 2014-11-20 at 07:41 -0800, Segher Boessenkool wrote: > An sh compiler fails to build on systems that have plain char unsigned. > Fix that. Ouch. Thanks for spotting this. OK for trunk, 4.9 and 4.8. Cheers, Oleg

[PATCH] sh: char isn't signed

2014-11-20 Thread Segher Boessenkool
An sh compiler fails to build on systems that have plain char unsigned. Fix that. 2014-11-20 Segher Boessenkool gcc/ PR target/60111 * config/sh/sh.c: Use signed char for signed field. --- gcc/config/sh/sh.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a

Re: "gimple-classes-v2-option-3" git branch committed to svn trunk as r217787

2014-11-20 Thread Michael Matz
Hi, On Thu, 20 Nov 2014, Richard Biener wrote: > > I don't think this API will make the non-C++-fans happier; I think the > > objection to the work I just merged is that it's adding more C++ than > > those people are comfortable with. > > How so? It's already super-ugly in those views. We deci

[PATCH][doc] Document cortex-a17 and cortex-a17.cortex-a7 -m{cpu,tune} options

2014-11-20 Thread Kyrill Tkachov
Hi all, As Joseph reminded, new -mcpu options should be documented in invoke.texi. This adds the documentation for the cortex-a17 and cortex-a17.cortex-a7 values. Ok to go in if the corresponding support patches posted earlier are accepted? Thanks, Kyrill 2014-11-19 Kyrylo Tkachov *

Re: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-11-20 Thread Chen Gang
Oh, sorry, after ran more than 10 days, the qemu crashed :-( After checked the output log, and compare with the original log, we know we have finished more than 90% test, and it is OK (no any new issues). I guess the reason is I started too many other things on this machine. Next, I shall try to

Re: [PATCH] gcc/ubsan.c: Extend 'pretty_name' space to avoid memory overflow

2014-11-20 Thread Chen Gang
On 11/17/14 18:52, Chen Gang wrote: > > What you said sound reasonable to me. > > I shall try to send patch v2 within this week (use pretty_printer). > > Thanks. > > On 11/17/14 16:15, Marek Polacek wrote: >> On Mon, Nov 17, 2014 at 08:38:19AM +0100, Jakub Jelinek wrote: >> >>> I think easiest

Re: [PATCH x86, PR60451] Expand even/odd permutation using pack insn.

2014-11-20 Thread Evgeny Stupachenko
Good point! "gen_shift" also requires only SSE2. That way we can optimize out interleave sequence for V16QI mode in expand_vec_perm_even_odd_1. Thanks! Evgeny Updated patch: diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c index 085eb54..054089b 100644 --- a/gcc/config/i386/i386.c ++

Re: [COMMITTED 1/3] Make TARGET_STATIC_CHAIN allow a function type

2014-11-20 Thread Richard Henderson
On 11/19/2014 08:56 PM, H.J. Lu wrote: > On Wed, Nov 19, 2014 at 10:04 AM, Jakub Jelinek wrote: >> On Wed, Nov 19, 2014 at 03:58:50PM +0100, Richard Henderson wrote: >>> As opposed to always being a decl. This is a prerequisite >>> to allowing the static chain to be loaded for indirect calls. >>>

[Ada] Improvements to handling of unchecked union discriminants

2014-11-20 Thread Arnaud Charlet
This patch avoids issuing a warning for a "missing" component clause for a discriminant in an unchecked union, and also avoids printing a line for such a component in the -gnatR2 output. The following program: 1. with Interfaces; 2. procedure Test_Union is 3. type Test_Type (Flag

Re: [PATCH] PR jit/63969: Fix segfault in error-handling when driver isn't found

2014-11-20 Thread David Malcolm
On Wed, 2014-11-19 at 23:09 -0800, Mike Stump wrote: > On Nov 19, 2014, at 10:23 PM, David Malcolm > wrote: > > It's not clear to me if I can approve my own patches to the jit > > So, to answer that, we look at MAINTAINERS, and look up your name: > > Various Maintainers >

Re: [PATCH] driver: ignore SIGINT while waiting on subprocesses to finish

2014-11-20 Thread Michael Matz
Hi, On Thu, 20 Nov 2014, Patrick Palka wrote: > > -wrapper is specifically also for invoking cc1 with gdb from the > > driver (that's the usecase documented with -wrapper), so it better > > should work as intended. I don't know what problems Patrick had with > > that, though. For me gcc -wra

Re: [AArch64, Patch] Add range-check for Symbol + offset addressing.

2014-11-20 Thread Tejas Belagod
On 14/11/14 17:33, Marcus Shawcroft wrote: On 14 November 2014 08:12, Tejas Belagod wrote: 2014-11-14 Tejas Belagod gcc/ * config/aarch64/aarch64-protos.h (aarch64_classify_symbol): Fixup prototype. * config/aarch64/aarch64.c (aarch64_expand_mov_immediate,

[Ada] Debugging information for inlined predefined units

2014-11-20 Thread Arnaud Charlet
The compiler suppresses debugging information on predefined units that are inlined in the code, because stepping into run-time units often complicates debugging activity. We make an exception for calls that appear in the source, when the unit is part of the Ada hierarchy, to facilitate monitoring

Re: [PATCH x86, PR60451] Expand even/odd permutation using pack insn.

2014-11-20 Thread Richard Henderson
On 11/20/2014 12:36 PM, Evgeny Stupachenko wrote: > + /* Required for "pack". */ > + if (!TARGET_SSE4_2 || d->one_operand_p) > +return false; Why the SSE4_2 check here when... > + > + /* Only V8HI, V16QI, V16HI and V32QI modes are more profitable than general > + shuffles. */ > + if

[Ada] Type conversion to String causes Constraint_Error

2014-11-20 Thread Arnaud Charlet
This patch modifies the mechanism which creates a subtype from an arbitrary expression. The mechanism now captures the bounds of all index constraints when the expression is of an array type. -- Source -- -- pack.ads with Ada.Finalization; use Ada.Finalization; packag

[Ada] Inter-unit inlining of expression functions with -gnatn1

2014-11-20 Thread Arnaud Charlet
This enables inter-unit inlining of expression functions with -gnatn1, or more simply with -O1/-O2 -gnatn. These functions are automatically candidates for inlining, but there were actually inlined across units only with -gnatn2, or more simply -O3 -gnatn. The following program must compile witho

[Ada] Spurious errors on extension aggregate for limited type

2014-11-20 Thread Arnaud Charlet
This patch fixes two errors in the handling of extension aggregates for limited types: Ancestor part of extension aggregate can itself be an extension aggregate as well as a function call that is rewritten as a reference. The following must compile quietly: gcc -c p2.adb gcc -c bugzilla.ads

[PATCH] Fix target/63977

2014-11-20 Thread Richard Henderson
My mistake yesterday. I thought I'd tested both x86_64 -m64/-m32, but not so. Anyway, as the comment says, the backend keeps querying the static chain, and if you don't early out, it sets ix86_static_chain_on_stack, at which point the setting is permanent and affects prologue generation, and not i

[PATCH] PR lto/63968: 175.vpr from cpu2000 fails to build with LTO

2014-11-20 Thread Martin Liška
Hello. As I reimplemented fibheap to C++ template, Honza told me that replace_key method actually supports just decrement operation. Old implementation suppress any feedback if we try to increase key: fibheap.c: ... /* If we wanted to, we could actually do a real increase by redeleting and

Re: LTO streaming of TARGET_OPTIMIZE_NODE

2014-11-20 Thread Bernd Schmidt
On 11/20/2014 02:20 PM, Richard Biener wrote: On Thu, 20 Nov 2014, Bernd Schmidt wrote: On 11/13/2014 05:06 AM, Jan Hubicka wrote: this patch adds infrastructure for proper streaming and merging of TREE_TARGET_OPTION. This breaks the offloading path via LTO since it introduces an incompatibi

Re: [PATCH] PR63426 Fix various signed integer overflows

2014-11-20 Thread David Edelsohn
On Thu, Nov 20, 2014 at 8:27 AM, Markus Trippelsdorf wrote: > Running the testsuite after bootstrap-ubsan on gcc112 shows several issues. > See > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63426 for the full list. > > This patch fixes several of them. > > Tested on powerpc64-unknown-linux-gnu.

Re: "gimple-classes-v2-option-3" git branch committed to svn trunk as r217787

2014-11-20 Thread Andrew MacLeod
On 11/20/2014 08:08 AM, Richard Biener wrote: On Thu, Nov 20, 2014 at 12:05 AM, Andrew MacLeod wrote: On 11/19/2014 05:24 PM, David Malcolm wrote: On Wed, 2014-11-19 at 22:36 +0100, Richard Biener wrote: On November 19, 2014 10:09:56 PM CET, Andrew MacLeod wrote: On 11/19/2014 03:43 PM, Ric

Re: [PATCH x86, PR60451] Expand even/odd permutation using pack insn.

2014-11-20 Thread Evgeny Stupachenko
Thank you. Patch with proposed fixes: diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c index 085eb54..09c0057 100644 --- a/gcc/config/i386/i386.c +++ b/gcc/config/i386/i386.c @@ -48322,6 +48322,120 @@ expand_vec_perm_vpshufb2_vpermq_even_odd (struct expand_vec_perm_d *d) return true

RE: [PATCH, committed] Update Automake files

2014-11-20 Thread Bernd Edlinger
Hello Jan-Benedict, > Hi! > > This patch updates the files taken from Automake. Committed. > > MfG, JBG the updated version of "missing" will confuse the gmp-4.3.2 configure script if it is installed in-tree with "contrib/download_prerequisites" and flex is not installed: ... checking readli

[PATCH] PR63426 Fix various signed integer overflows

2014-11-20 Thread Markus Trippelsdorf
Running the testsuite after bootstrap-ubsan on gcc112 shows several issues. See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63426 for the full list. This patch fixes several of them. Tested on powerpc64-unknown-linux-gnu. OK for trunk? Thanks. 2014-11-20 Markus Trippelsdorf * conf

Re: LTO streaming of TARGET_OPTIMIZE_NODE

2014-11-20 Thread Richard Biener
On Thu, 20 Nov 2014, Bernd Schmidt wrote: > On 11/13/2014 05:06 AM, Jan Hubicka wrote: > > this patch adds infrastructure for proper streaming and merging of > > TREE_TARGET_OPTION. > > This breaks the offloading path via LTO since it introduces an incompatibility > in LTO format between host and

Re: [PATCH] Disable an unsafe VRP transformation when -fno-strict-overflow is set

2014-11-20 Thread Richard Biener
On Thu, Nov 20, 2014 at 4:21 AM, Patrick Palka wrote: > VRP may simplify a conditional like i <= 5 to i == 5 if it is known that > the lower bound of i's range is 5, e.g. [5, +INF]. But if the upper > bound of i's range is also overflow infinity, i.e. [5, +INF(OVF)] then > this transformation is

Re: bitmap fix for current

2014-11-20 Thread Richard Biener
On Thu, Nov 20, 2014 at 1:18 AM, Mike Stump wrote: > On Nov 14, 2014, at 2:26 AM, Richard Biener > wrote: >> On Fri, Nov 14, 2014 at 2:10 AM, Jeff Law wrote: >>> On 11/13/14 12:37, Mike Stump wrote: I was doing a merge, and it failed to even compile the runtime libraries due to c

Re: "gimple-classes-v2-option-3" git branch committed to svn trunk as r217787

2014-11-20 Thread Richard Biener
On Wed, Nov 19, 2014 at 11:24 PM, David Malcolm wrote: > On Wed, 2014-11-19 at 22:36 +0100, Richard Biener wrote: >> On November 19, 2014 10:09:56 PM CET, Andrew MacLeod >> wrote: >> >On 11/19/2014 03:43 PM, Richard Biener wrote: >> >> On November 19, 2014 8:26:23 PM CET, Andrew MacLeod >> > wro

Re: "gimple-classes-v2-option-3" git branch committed to svn trunk as r217787

2014-11-20 Thread Richard Biener
On Thu, Nov 20, 2014 at 12:05 AM, Andrew MacLeod wrote: > On 11/19/2014 05:24 PM, David Malcolm wrote: >> >> On Wed, 2014-11-19 at 22:36 +0100, Richard Biener wrote: >>> >>> On November 19, 2014 10:09:56 PM CET, Andrew MacLeod >>> wrote: On 11/19/2014 03:43 PM, Richard Biener wrote: >>>

Re: OpenACC middle end changes

2014-11-20 Thread Bernd Schmidt
On 11/20/2014 07:52 AM, Jakub Jelinek wrote: On Thu, Nov 20, 2014 at 03:19:11AM +0100, Bernd Schmidt wrote: Thomas had apparently already pointed out an issue with the new gomp_target class (there are multiple similar types of statements we want to handle with OpenACC, they have different codes

  1   2   >