Re: [PATCH v2] gcc/c-family/c-cppbuiltin.c: Let buffer enough to print host wide integer value

2014-11-24 Thread Joseph Myers
On Sun, 23 Nov 2014, Chen Gang wrote: > + gcc_assert (wi::fits_to_tree_p(value, integer_type_node)); Watch formatting: space before '(' in the wi::fits_to_tree_p call. Applies elsewhere in this patch as well. When making such an interface change, (a) you should update the comment on builtin_

[PATCH] Add verify_sese

2014-11-24 Thread Tom de Vries
Richard, I ran into a problem with my oacc kernels directive patch series where tail-merge added another entry into a region that was previously single-entry-single-exit. That resulted in hitting this assert in calc_dfs_tree: ... /* This aborts e.g. when there is _no_ path from ENTRY to EXI

Re: patch to fix PR63897

2014-11-24 Thread H.J. Lu
On Fri, Nov 21, 2014 at 1:32 PM, Vladimir Makarov wrote: > The following patch fixes PR63897. The details can be found on > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63897 > > The patch was successfully bootstrapped and tested on x86 and x86-64. > > Committed as rev. 217947. > > 2014-11-21

Re: [PATCH v2] gcc/c-family/c-cppbuiltin.c: Let buffer enough to print host wide integer value

2014-11-24 Thread Chen Gang
OK, thank you for your work, I shall send patch v3 for it. Send from Lenovo A788t. Joseph Myers wrote: >On Sun, 23 Nov 2014, Chen Gang wrote: > >> + gcc_assert (wi::fits_to_tree_p(value, integer_type_node)); > >Watch formatting: space before '(' in the wi::fits_to_tree_p call. >Applies elsew

Re: [PATCH v3] Use strtoll instead of atoll in gcov-tool.c

2014-11-24 Thread John David Anglin
On 15-Sep-14, at 5:40 AM, Richard Biener wrote: Let's wait for other opinions. We now have implementations of strtol and strtoll in libiberty. So, it's better to use these than atoll. As suggested by Jakub, the patch now falls back to int64_t if the host doesn't have long long. Tested

Re: [RFC] First steps towards segregating types.

2014-11-24 Thread Joseph Myers
On Mon, 24 Nov 2014, Richard Biener wrote: > > TREE_LIST should die (with the typical replacement being vec); > > most lists do not need all the overhead of individually allocated objects > > with (code, flags, type, chain, value, purpose). Probably TREE_VEC too. > > Note that there is nothing w

Re: [Patch] Improving jump-thread pass for PR 54742

2014-11-24 Thread Sebastian Pop
Sebastian Pop wrote: > I removed the return -1 and started a bootstrap on powerpc64-linux. Bootstrap passed on top of the 4 previous patches on powerpc64-linux. > I will report the valgrind output. The output from valgrind looks closer to the output of master with no other patches: still 1M more

Re: [PATCH][AArch64] Remove crypto extension from default for cortex-a53, cortex-a57

2014-11-24 Thread Gerald Pfeifer
On Tuesday 2014-11-18 09:38, Kyrill Tkachov wrote: Here's what I propose. + The cryptographic extensions to the ARMv8-A architecture are no + longer enabled by default when specifying the + -mcpu=cortex-a53, -mcpu=cortex-a57 or + -mcpu=cortex-a57.cortex-a53 options. To e

Re: [C PATCH] Fix PR63877

2014-11-24 Thread Joseph Myers
On Mon, 24 Nov 2014, Marek Polacek wrote: > This PR is basically the same as PR54113, except this time it's about > -Wmissing-declarations and not about -Wmissing-prototypes. The problem > here is that we were emitting bogus warning for a correct use of an > inline function. Thus fixed in the sa

[PATCH] rs6000: Remove iorxor/IORXOR code attrs

2014-11-24 Thread Segher Boessenkool
As Richard pointed out, those do nothing more than code/CODE. Tested etc.; okay for trunk? Segher 2014-11-21 Segher Boessenkool gcc/ * config/rs6000/rs6000.md (iorxor, IORXOR): Delete code_attrs. (rest of file): Replace those with code resp. CODE. --- gcc/config/rs6000/rs6

[PATCH] rs6000: Replace a stray addic with addi

2014-11-24 Thread Segher Boessenkool
Tested as usual... okay for trunk? Segher 2014-11-24 Segher Boessenkool gcc/ * config/rs6000/sysv4.h (ASM_OUTPUT_REG_POP): Use addi instead of addic. --- gcc/config/rs6000/sysv4.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/config/rs6000/sysv4.

[PATCH] mn10300: Fix an ICE

2014-11-24 Thread Segher Boessenkool
`lcc' is not an insn but just a pattern. This caused a build error in libgcc. Tested with a cross compiler build (which fails without and succeeds with the patch). Not tested much more; this compiler really likes to ICE, something with ipa-icf. Is this okay for trunk? Segher 2014-11-24 Segh

[PATCH] crtstuff: Add missing semicolon

2014-11-24 Thread Segher Boessenkool
I wonder how this survived so long, I must be building some strange configs (it failed on an avr cross). Okay for trunk? Segher 2014-11-24 Segher Boessenkool libgcc/ * crtstuff.c (__do_glbal_ctors_1): Add missing semicolon. --- libgcc/crtstuff.c | 2 +- 1 file changed, 1 insertio

[PATCH] Fix a bug in bt-load.c

2014-11-24 Thread Segher Boessenkool
This caused ICEs on sh64. `min_cost' and `def' here are supposed to refer to the same element; removing it from the heap before asking the heap for the key doesn't work (and at the end of the loop here we will ask for min_key on an empty heap, which then does gcc_unreachable). Bootstrapped and te

Re: [Patch] Improving jump-thread pass for PR 54742

2014-11-24 Thread Jeff Law
On 11/24/14 18:09, Sebastian Pop wrote: Sebastian Pop wrote: I removed the return -1 and started a bootstrap on powerpc64-linux. Bootstrap passed on top of the 4 previous patches on powerpc64-linux. I will report the valgrind output. The output from valgrind looks closer to the output of m

[Patch, Fortran] convert almost all {warning,error}_now to common diagnostic

2014-11-24 Thread Tobias Burnus
With this patch – and the still pending patch for gfc_internal_error at https://gcc.gnu.org/ml/gcc-patches/2014-11/msg03003.html – all error/warning messages have been converted to the common diagnostic, except for: (a) those majority which might need buffering (gfc_error, gfc_warning); (b) th

Re: [PATCH] Fix find_base_term in 32-bit -fpic code (PR lto/64025)

2014-11-24 Thread Uros Bizjak
On Tue, Nov 25, 2014 at 12:25 AM, Jakub Jelinek wrote: > The fallback delegitimization I've added as last option mainly for > debug info purposes, when we don't know if the base is a PIC register > or say a PIC register plus some addend, unfortunately in some tests > broke find_base_term, which f

[PATCH] Fix PR64059

2014-11-24 Thread Markus Trippelsdorf
This fixes PR64059 where a call to get_dynamic_type wasn't guarded by flag_devirtualize. Tested on powerpc64-unknown-linux-gnu. Preapproved by Honza on bugzilla. Commited to trunk. PR ipa/64059 * ipa-prop.c (ipa_analyze_call_uses): Don't call get_dynamic_type when devirtua

<    1   2