[SH] Cleanup endianness macros

2013-10-26 Thread Oleg Endo
Hello, The attached patch adds a new macro TARGET_BIG_ENDIAN. I think it's a bit easier to read than !TARGET_LITTLE_ENDIAN. Moreover, some not so obvious looking uses of TARGET_LITTLE_ENDIAN for selecting MSW and LSW register offsets are clarified by using new macros SH_REG_MSW_OFFSET and SH_REG

Re: [PATCH] Enhance ifcombine to recover non short circuit branches

2013-10-26 Thread Andrew Pinski
On Sat, Oct 26, 2013 at 2:30 PM, Andrew Pinski wrote: > On Fri, Oct 18, 2013 at 2:21 AM, Zhenqiang Chen > wrote: >> On 18 October 2013 00:58, Jeff Law wrote: >>> On 10/17/13 05:03, Richard Biener wrote: >> >> Is it OK for trunk? > > > I had a much simpler change which did bas

[SH, committed] Fix some formatting

2013-10-26 Thread Oleg Endo
Hello, I've just committed the attached obvious patch as rev 204098. Tested with make all-gcc. Cheers, Oleg gcc/ChangeLog: * config/sh/sh.md (movmemsi): Remove empty constraints and predicates. Fix formatting. (cmpstr_t, cmpstrsi): Fix formatting. Index: gcc/config/sh/

Re: [libiberty] Fix testsuite/test-demangle.c

2013-10-26 Thread Ian Lance Taylor
On Sat, Oct 26, 2013 at 12:48 PM, Gerald Pfeifer wrote: > > 2013-10-26 Gerald Pfeifer > > * testsuite/test-demangle.c: Include unistd.h. This is OK. Thanks. Ian

Re: [PATCH] Enhance ifcombine to recover non short circuit branches

2013-10-26 Thread Andrew Pinski
On Wed, Oct 16, 2013 at 2:12 AM, Zhenqiang Chen wrote: > Hi, > > The patch enhances ifcombine pass to recover some non short circuit > branches. Basically, it will do the following transformation: > > Case 1: > if (cond1) > if (cond2) > ==> > if (cond1 && cond2) > > Case 2: > if (cond1)

Re: Minor mudflap fallout

2013-10-26 Thread Hans-Peter Nilsson
On Sat, 26 Oct 2013, Andrew Pinski wrote: > I think you could use PRED_NORETURN which should be a reasonable replacement. I've totally missed that one, thanks. brgds, H-P

Re: [PATCH] Enhance ifcombine to recover non short circuit branches

2013-10-26 Thread Andrew Pinski
On Fri, Oct 18, 2013 at 2:21 AM, Zhenqiang Chen wrote: > On 18 October 2013 00:58, Jeff Law wrote: >> On 10/17/13 05:03, Richard Biener wrote: > > Is it OK for trunk? I had a much simpler change which did basically the same from 4.7 (I can update it if people think thi

Re: [PATCH] Final removal of mudflap

2013-10-26 Thread Joseph S. Myers
As far as I can see, the commit left empty libmudflap directories around rather than removing them (SVN, unlike git, allows empty directories to be represented in the repository). -- Joseph S. Myers jos...@codesourcery.com

Re: Minor mudflap fallout

2013-10-26 Thread Jeff Law
On 10/26/13 05:19, Paolo Carlini wrote: Hi Marek Polacek ha scritto: On Sat, Oct 26, 2013 at 12:46:41PM +0200, Paolo Carlini wrote: ... seriously, in libmudflap.c++, 41, etc But aren't all these removed in commit 204090? Ah! I didn't notice something such aggressive was going on! Excell

Re: Minor mudflap fallout

2013-10-26 Thread Jeff Law
On 10/26/13 07:45, Hans-Peter Nilsson wrote: On Sat, 26 Oct 2013, Jeff Law wrote: It appears that mudflap creeped into one additional file (targhooks) between the time I bootstrapped the final change and committed the change. This also elimiantes PRED_MUDFLAP which I missed the first time aro

[libiberty] Fix testsuite/test-demangle.c

2013-10-26 Thread Gerald Pfeifer
This is similar to my fix for testsuite/test-expandargv.c last week. I'm still not sure why this does not trigger on GNU/Linux distributions, or earlier version of FreeBSD. On FreeBSD 10 one does need to #include to get getpagesize which is used by the test. With this, the test now compiles on

Re: [committed] Dump OpenMP version number listed for -fopenmp at doc/invoke.texi

2013-10-26 Thread Gerald Pfeifer
On Fri, 25 Oct 2013, Tobias Burnus wrote: > I have committed the patch below as obvious (Rev. 204049). > Seemingly, there are a lot of places which have to be updated > for a new OpenMP version ... One way to address that is _not_ to refer to concrete versions unless specifically required. This

Re: Minor mudflap fallout

2013-10-26 Thread Andrew Pinski
On Sat, Oct 26, 2013 at 6:45 AM, Hans-Peter Nilsson wrote: > On Sat, 26 Oct 2013, Jeff Law wrote: > >> It appears that mudflap creeped into one additional file (targhooks) between >> the time I bootstrapped the final change and committed the change. This >> also >> elimiantes PRED_MUDFLAP which

Fixing cris-* breakage (was: Minor mudflap fallout)

2013-10-26 Thread Hans-Peter Nilsson
On Sat, 26 Oct 2013, Hans-Peter Nilsson wrote: > Yeah, cris-elf is broken too. I use PRED_MUDFLAP in > cris_emit_trap_for_misalignment, which should explain its use. > Is there a replacement? Actually looking at predict.def, I see PRED_COLD_LABEL, which wasn't there when I worked with this code bu

Re: Aliasing: look through pointer's def stmt

2013-10-26 Thread Marc Glisse
On Fri, 25 Oct 2013, Marc Glisse wrote: On Fri, 25 Oct 2013, Richard Biener wrote: you can followup with handling POINTER_PLUS_EXPR if you like. Like this? (bootstrap+testsuite on x86_64-unknown-linux-gnu) 2013-10-26 Marc Glisse gcc/ * tree-ssa-alias.c (ao_ref_init_from_ptr_and_

reverting a LRA change

2013-10-26 Thread Vladimir Makarov
It seems one my yesterday patch created problems with mode switching: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58679 I reverted it until further investigation (committed as rev. 204094). Sorry for inconvenience. 2013-10-26 Vladimir Makarov Revert: 2013-10-25 Vladimir

Re: Minor mudflap fallout

2013-10-26 Thread Hans-Peter Nilsson
On Sat, 26 Oct 2013, Jeff Law wrote: > It appears that mudflap creeped into one additional file (targhooks) between > the time I bootstrapped the final change and committed the change. This also > elimiantes PRED_MUDFLAP which I missed the first time around. > > Given this is currently breaking

[C11-atomic] Miscellaneous fixes 5/n

2013-10-26 Thread Joseph S. Myers
I've applied this small patch to C11-atomic branch to fix ICEs that could occur with C_MAYBE_CONST_EXPR through lack of folding. c: 2013-10-26 Joseph Myers * c-typeck.c (build_atomic_assign): Fold RHS if not already folded. testsuite: 2013-10-26 Joseph Myers * gcc.

Re: Minor mudflap fallout

2013-10-26 Thread Paolo Carlini
Hi Marek Polacek ha scritto: >On Sat, Oct 26, 2013 at 12:46:41PM +0200, Paolo Carlini wrote: >> ... seriously, in libmudflap.c++, 41, etc > >But aren't all these removed in commit 204090? Ah! I didn't notice something such aggressive was going on! Excellent! Thanks, Paolo

Re: Minor mudflap fallout

2013-10-26 Thread Marek Polacek
On Sat, Oct 26, 2013 at 12:46:41PM +0200, Paolo Carlini wrote: > ... seriously, in libmudflap.c++, 41, etc But aren't all these removed in commit 204090? Marek

Re: Minor mudflap fallout

2013-10-26 Thread Paolo Carlini
... seriously, in libmudflap.c++, 41, etc Paolo

Re: Minor mudflap fallout

2013-10-26 Thread Paolo Carlini
Marek Polacek ha scritto: >Which ones? All mudflap tests should be gone now. Just browse gcc-testresults ;) Paolo

Re: Minor mudflap fallout

2013-10-26 Thread Marek Polacek
On Sat, Oct 26, 2013 at 12:31:34PM +0200, Paolo Carlini wrote: > ... slightly out of topic, sorry Jeff, but your message made me think that we > should probably do something about the mudflap fails that we have got since > forever (at least on Linux): to your best knowledge is somebody actively

Re: Minor mudflap fallout

2013-10-26 Thread Paolo Carlini
... slightly out of topic, sorry Jeff, but your message made me think that we should probably do something about the mudflap fails that we have got since forever (at least on Linux): to your best knowledge is somebody actively working on those? Should be just xfails for now?!? Thanks! Paolo

Minor mudflap fallout

2013-10-26 Thread Jeff Law
It appears that mudflap creeped into one additional file (targhooks) between the time I bootstrapped the final change and committed the change. This also elimiantes PRED_MUDFLAP which I missed the first time around. Given this is currently breaking boostrap, I'm checking it in proactively

Re: [Patch] Refactor regex executors

2013-10-26 Thread Paolo Carlini
On 10/26/2013 01:35 AM, Tim Shen wrote: On Fri, Oct 25, 2013 at 10:27 AM, Paolo Carlini wrote: Uhm, cute, didn't consider the naming issue, but I'm not sure we want to do that... What about having the code in a split.h and including it from split.cc and split_bfs.cc instead? Sorry for late. P

vzeroupper mode-switching pass regression (Was: Re: patch adding LRA usage for ppc)

2013-10-26 Thread Uros Bizjak
Hello! > 2013-10-25 Vladimir Makarov > > ... > * lra-spills.c (lra_final_code_change): Remove useless move insns. This change regressed x86 AVX vzeroupper insertion pass and exposed another instance of PR 58679 [1]. The testcase: --cut here-- typedef long long __m128i __attribute__ (

free is a killer

2013-10-26 Thread Marc Glisse
Hello, this patch teaches gcc that free kills the memory its argument points to. The equality test is probably too strict, I guess we can loosen it later (unless you have suggestions?). Note that the corresponding code for BUILT_IN_MEMCPY and others seems suspicious to me, it looks like it i