[SPARC] Fix PR target/52717

2012-04-09 Thread Eric Botcazou
This is the LTO issue with -fpic/-fpie on SPARC/Linux. When the special GOT helper function is generated, the compiler invokes: make_decl_one_only (decl, DECL_ASSEMBLER_NAME (decl)); on it, which sets the TREE_PUBLIC flag as expected. The problem is that, in LTO mode, DECL_ASSEMBLER

[SPARC] Adjust testsuite parameterization

2012-04-09 Thread Eric Botcazou
This both eliminates bogus failures in the testsuite on SPARC64/Linux when you configure --with-cpu=v8 and enables more tests on SPARC/Solaris (which is by default configured --with-cpu=v9). Tested on SPARC64/Linux and SPARC/Solaris, applied on mainline and 4.7 branch. 2012-04-09 Eric Botcazo

[patch] Emit variable as size attribute in debug info

2012-04-09 Thread Eric Botcazou
Hi, in Ada record (structure) types can have a variable size and, in this case, the debug info contains -1 as size attribute for the type. While the long term plan is to support general expressions as size attribute, a small step forward is to accept simple variables, which is an important sub

[RFC] Unconditionally clean up CFG before emitting prologue

2012-04-09 Thread Eric Botcazou
Hi, with the numerous checks generated in Ada, the gimplification process can generated dead branches that aren't easily eliminated at -O0 and can impair the debugging experience. We have found that unconditionally cleaning up the CFG before emitting the prologue/epilogue can help in some case

[PATCH, i386]: Fix PR 52883, ICE in simplify_const_unary_operation, at simplify-rtx.c:1464

2012-04-09 Thread Uros Bizjak
Hello! As pointed out by Eric, there is this little detail in the documentation w.r.t sign and zero-extending conversions: "For all conversion operations, X must not be `VOIDmode' because the mode in which to do the conversion would not be known. The conversion must either be done at compile-tim

[C/ObjC/C++/ObjC++] cleanup diagnostics initialization

2012-04-09 Thread Manuel López-Ibáñez
This patch cleans up the diagnostic initialization of the C-family FEs. It keeps the default of no line-wrapping. It moves together all initializations and deletes code that has no effect. Bootstrapped and regression tested on x86-unknown-linux-gnu with enable-languages=all,ada,obj-c++. OK? 201

Re: [PATCH] Fix PR52614

2012-04-09 Thread Richard Guenther
On Fri, Apr 6, 2012 at 12:15 AM, Ian Lance Taylor wrote: > Richard Guenther writes: > >> On Thu, Apr 5, 2012 at 6:22 AM, Mike Stump wrote: >>> On Apr 4, 2012, at 7:56 PM, William J. Schmidt wrote: There seems to be tacit agreement that the vector tests should use -fno-common on all tar

Re: [C/ObjC/C++/ObjC++] cleanup diagnostics initialization

2012-04-09 Thread Gabriel Dos Reis
On Mon, Apr 9, 2012 at 5:12 AM, Manuel López-Ibáñez wrote: > This patch cleans up the diagnostic initialization of the C-family > FEs. It keeps the default of no line-wrapping. It moves together all > initializations and deletes code that has no effect. > > Bootstrapped and regression tested on x8

Re: [C11-atomic] [patch] gimple atomic statements

2012-04-09 Thread Richard Guenther
On Fri, Apr 6, 2012 at 10:13 AM, Richard Sandiford wrote: > Richard Guenther writes: >>> They can affect shared memory in some ways like a call, but don't have many >>> of the other attributes of call.  They are really more like an assignment or >>> other operation with arbitrary shared memory si

Re: [PR51570] minimize ENTRY_VALUEs in debug location lists

2012-04-09 Thread Jakub Jelinek
On Mon, Apr 09, 2012 at 02:50:02AM -0300, Alexandre Oliva wrote: > for gcc/ChangeLog > from Alexandre Oliva > > PR debug/51570 > * var-tracking.c (expand_depth): New type. > (onepart_aux, expand_loc_callback_data): Change depth type to it. > (loc_exp_dep_alloc): Adjust

Re: [PATCH] Fix PR tree-optimization/52870 (another crash in SLP pattern detection)

2012-04-09 Thread Richard Guenther
On Fri, Apr 6, 2012 at 8:30 PM, Ulrich Weigand wrote: > Hello, > > PR 52870 is another crash in vectorizer pattern detection that was uncovered > by Ira's patch to enable patterns for SLP as well. > > In this case, the problem is that vect_recog_widen_mult_pattern detects a > statement as part of

Re: [PR 50826] [DF] no debug temps for artificial uses

2012-04-09 Thread Jakub Jelinek
On Mon, Apr 09, 2012 at 03:29:05AM -0300, Alexandre Oliva wrote: > from Alexandre Oliva > > PR debug/50826 > * df-problems.c (df_note_bb_compute): Do not take note of > debug uses for whose REGs we won't emit DEAD or UNUSED notes. > Index: gcc/df-problems.c >

Re: [RFC] Unconditionally clean up CFG before emitting prologue

2012-04-09 Thread Richard Guenther
On Mon, Apr 9, 2012 at 11:55 AM, Eric Botcazou wrote: > Hi, > > with the numerous checks generated in Ada, the gimplification process can > generated dead branches that aren't easily eliminated at -O0 and can impair > the debugging experience.  We have found that unconditionally cleaning up the >

Re: [C/ObjC/C++/ObjC++] cleanup diagnostics initialization

2012-04-09 Thread Manuel López-Ibáñez
On 9 April 2012 12:43, Gabriel Dos Reis wrote: > On Mon, Apr 9, 2012 at 5:12 AM, Manuel López-Ibáñez > wrote: >>        * c-common.h (c_common_initialize_diagnostics): Likewise. > > Make the comment less personal; we don't who "I" is in "I'm putting them here" > in three months (nor should we hav

Re: fix left-over debug insns in DCE

2012-04-09 Thread Eric Botcazou
> Some more context here: this enables DCE to turn removed insns into > debug temps when they're useful for debug info. It further improves > debug info quality when installed along with the patch I just posted for > PR 48866. Without it, a number of chains of debug temps that lead to a > real in

Re: [wwwdocs] Minor tweak to release branch status descriptions

2012-04-09 Thread Gerald Pfeifer
Release managers, On Sun, 11 Sep 2011, Gerald Pfeifer wrote: > I'd like to make the change below which helps with lower resolution > systems (or non-maximized web browsers) or larger font sizes. > > Practically I think everyone would understand the new version is the > same as the old, and the of

[wwwdocs] Rotate news

2012-04-09 Thread Gerald Pfeifer
Applied. Gerald Index: index.html === RCS file: /cvs/gcc/wwwdocs/htdocs/index.html,v retrieving revision 1.848 diff -u -3 -p -r1.848 index.html --- index.html 22 Mar 2012 09:58:00 - 1.848 +++ index.html 9 Apr 2012 14:01:29

Re: Mirror gcc/contrib -> src/contrib? [was Re: [patch, gcc RFA] dg-extract-results.sh: Handle KFAILs.]

2012-04-09 Thread Tom Tromey
> "Doug" == Doug Evans writes: Doug> On Thu, Mar 15, 2012 at 11:54 AM, Mike Stump wroteDoug> btw, is there any interest in mirroring gcc/contrib -> src/contrib? Doug> [instead of gdb having its own copy in gdb/testsuite, it *could* use Doug> the one in, e.g., src/contrib. I don't have a str

Re: Mirror gcc/contrib -> src/contrib? [was Re: [patch, gcc RFA] dg-extract-results.sh: Handle KFAILs.]

2012-04-09 Thread Doug Evans
On Mon, Apr 9, 2012 at 7:43 AM, Tom Tromey wrote: >> "Doug" == Doug Evans writes: > > Doug> On Thu, Mar 15, 2012 at 11:54 AM, Mike Stump > wroteDoug> btw, is there any interest in mirroring gcc/contrib -> src/contrib? > Doug> [instead of gdb having its own copy in gdb/testsuite, it *could*

Re: Mirror gcc/contrib -> src/contrib? [was Re: [patch, gcc RFA] dg-extract-results.sh: Handle KFAILs.]

2012-04-09 Thread Gerald Pfeifer
On Fri, 6 Apr 2012, Doug Evans wrote: > btw, is there any interest in mirroring gcc/contrib -> src/contrib? > [instead of gdb having its own copy in gdb/testsuite, it *could* use > the one in, e.g., src/contrib. I don't have a strong opinion. I'm > just looking for others opinions, as the topic ca

Re: [RFC] Unconditionally clean up CFG before emitting prologue

2012-04-09 Thread Eric Botcazou
> Isn't the gimple cfg-cleanup we run post optimization (right before > expansion) not enough? Or the cfg-cleanup we perform right after expansion > now? At least if the branches are really caused by the gimplification > process I would expect things to be cleaned up at this point, no? At least t

[Patch, Fortran, F03] PR52909: Procedure pointers not private to modules

2012-04-09 Thread Janus Weil
Hi all, here is a simple patch for a problem recently reported by Andrew Benson: Procedure pointers inside a module don't receive the proper name mangling, which can lead to name collisions (as the test case demonstrates). The patch fixes this and regtests cleanly on x86_64-unknown-linux-gnu. I a

Re: Support for Runtime CPU type detection via builtins (issue5754058)

2012-04-09 Thread Sriraman Tallam
On Sun, Apr 8, 2012 at 6:17 AM, Gerald Pfeifer wrote: > On Thu, 29 Mar 2012, Sriraman Tallam wrote: >> Hi, >> >>     I have made a new patch to only have two builtins : >> >> * __builtin_cpu_is ("") >> * __builtin_cpu_supports ("") >> >> apart from the cpu init builtin, __builtin_cpu_init. > > I d

Re: [C/ObjC/C++/ObjC++] cleanup diagnostics initialization

2012-04-09 Thread Gabriel Dos Reis
On Mon, Apr 9, 2012 at 6:19 AM, Manuel López-Ibáñez wrote: > On 9 April 2012 12:43, Gabriel Dos Reis wrote: >> On Mon, Apr 9, 2012 at 5:12 AM, Manuel López-Ibáñez >> wrote: >>>        * c-common.h (c_common_initialize_diagnostics): Likewise. >> >> Make the comment less personal; we don't who "I"

Re: [wwwdocs] Minor tweak to release branch status descriptions

2012-04-09 Thread Richard Guenther
On Mon, Apr 9, 2012 at 3:47 PM, Gerald Pfeifer wrote: > Release managers, > > On Sun, 11 Sep 2011, Gerald Pfeifer wrote: >> I'd like to make the change below which helps with lower resolution >> systems (or non-maximized web browsers) or larger font sizes. >> >> Practically I think everyone would

Fix partitioning of aliases

2012-04-09 Thread Jan Hubicka
Hi, this patch fixes several different ICEs related to handling aliases in WHOPR partitioning. It took me over week debug this, but when variable alias is added to a boundary and its destination is not added, we get queue of unforutnate events where the destinatoin gets analyzed and added at ltran

Re: [v3] fix libstdc++/52476

2012-04-09 Thread François Dumont
Attached patch applied to 4_7-branch. 2012-04-09 François Dumont PR libstdc++/52476 * include/bits/hashtable.h (_Hashtable<>::_M_rehash_aux): Add. (_Hashtable<>::_M_rehash): Use the latter. * testsuite/23_containers/unordered_multimap/insert/52476.cc: New.

Re: [PATCH] Caret diagnostics

2012-04-09 Thread Manuel López-Ibáñez
. (pp_base_get_prefix): New. * common.opt (fdiagnostics-show-caret): New option. * doc/invoke.texi (fdiagnostics-show-caret): Document it. testsuite/ * lib/prune.exp: Add -fno-diagnostics-show-caret. caret-diagnostics-20120409.diff Description: Binary data

Re: [PATCH, RFC] Extend __attribute__((format)) with user-specified conversion sequences

2012-04-09 Thread Eddie Kohler
Hi, thanks for the reply. On 4/8/12 6:25 AM, Joseph S. Myers wrote: Character pairs like this don't seem very extensible, in that you are providing meanings for any even-length string, rather than (for example) only a limited subset of strings leaving room for meanings to be assigned later to ot

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-09 Thread Adam Conrad
On Thu, Apr 05, 2012 at 10:50:50AM +1200, Michael Hope wrote: > On 4 April 2012 18:54, Jakub Jelinek wrote: > > > > If the agreement is that arm 32-bit softfp really needs to be installable > > alongside 32-bit hardfp (and alongside aarch64), then IMHO it should do it > > like all other multilib p

Re: [v3] fix libstdc++/52476

2012-04-09 Thread Jonathan Wakely
On 9 April 2012 20:25, François Dumont wrote: > > I don't think I have the necessary rights to close the PR on bugzilla, I > haven't been able to do so. You should be able to using your @gcc.gnu.org account. What if you assign it to yourself first?

Re: Propagate profile counts after switch case expansion (issue5896043)

2012-04-09 Thread Easwaran Raman
On Sun, Mar 25, 2012 at 9:40 PM, Easwaran Raman wrote: > On Sun, Mar 25, 2012 at 12:22 PM, Jan Hubicka wrote: >>> This patch propagates execution count of thee case labels of a >>> switch-case statement after its expansion. Bootstraps and all >>> tests pass. OK for trunk? >> >> Hi, >> while this

[patch] Add generic __builtin_bswap16 support

2012-04-09 Thread Eric Botcazou
Hi, this adds generic support for __builtin_bswap16 (only PowerPC has it for now). It is mapped to the bswap optab in HImode, whose implementation is as follows: - if a bswaphi2 pattern is present (PowerPC), it is directly used; or else - if a rotlhi2/rotrhi2 pattern is present (x86), it is di

Re: [SH] Fold big/little endian word code with MSW and LSW

2012-04-09 Thread Kaz Kojima
Oleg Endo wrote: >> On a second thought, it might be better to simplify those cases in >> the .md file, where low/high subregs are used by using 'gen_highpart' >> and 'gen_lowpart' for example, instead of checking the endianness over >> and over again. >> I'm trying out a couple of things... > >

Re: [PATCH] Caret diagnostics

2012-04-09 Thread Jason Merrill
On 04/09/2012 04:01 PM, Manuel López-Ibáñez wrote: * It uses the default cutoff as max_width, whatever it is (as controlled by -fmessage-length). * It uses the pretty-printer. The text cannot (should not) wrap because we still print only max_width chars at most. Hmm, I think if pp_line_cutoff

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-09 Thread Mike Frysinger
On Monday 09 April 2012 16:48:06 Adam Conrad wrote: > On Thu, Apr 05, 2012 at 10:50:50AM +1200, Michael Hope wrote: > > On 4 April 2012 18:54, Jakub Jelinek wrote: > > > If the agreement is that arm 32-bit softfp really needs to be > > > installable alongside 32-bit hardfp (and alongside aarch64),

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-09 Thread Adam Conrad
On Mon, Apr 09, 2012 at 07:14:45PM -0400, Mike Frysinger wrote: > > again, saying "/lib//" isn't multiarch is bunk. but it sounds > like you're fine with /libhf/, so there isn't anything left to thrash about > there. I appreciate your careful reading of my email and the issues I outlined, and

Re: [PATCH] Fix PR52571

2012-04-09 Thread Mike Stump
On Mar 14, 2012, at 5:59 AM, Richard Guenther wrote: >> This fixes PR52571, we should not align DECL_COMMON variables as >> they might be pre-empted by a definition with lower alignment. I added this to the PR: Ah, I had another thought. COMDAT and LINKONCE things I don't think can be realigned

[SH] PR 50751 - add HImode displacement addressing support

2012-04-09 Thread Oleg Endo
Hello, The attached patch adds HImode addressing support. Tested against rev. 186243 with sudo make -k check RUNTESTFLAGS="--target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2a/-mb,-m2a-single/-mb,-m4/-ml,-m4/-mb, -m4-single/-ml,-m4-single/-mb,-m4a-single/-ml,-m4a-single/-mb}" and no new failures. Test ca

[PATCH] Fix PR middle-end/52894

2012-04-09 Thread John David Anglin
The current 4.5, 4.6 and 4.7 branches are now seriously broken for all PA target due to the fix applied for PR middle-end/52640, an optimization fix. This is because the PA backend defers output of function descriptors and externals (hpux) using TARGET_ASM_FILE_END. As a result, assemble_external

PATCH: PR other/52777: [4.8 Regression] ./options.h:3546:0: error: "MASK_SIO" redefined [-Werror]

2012-04-09 Thread H.J. Lu
Hi, I checked in this patch, which has been approved in PR. H.J. --- Index: config/pa/pa.opt === --- config/pa/pa.opt(revision 186256) +++ config/pa/pa.opt(working copy) @@ -138,9 +138,3 @@ Generate PA1.1 code mspace-regs

execute permissions in ada

2012-04-09 Thread Mike Stump
I'd like to remove execute permissions for: gcc/ada/*.adb Ok?

remove execute permissions from gcc/testsuite

2012-04-09 Thread Mike Stump
Remove execute permissions from gcc/testsuite: * gcc.dg/dll-8.c: Remove execute permissions. * g++.dg/ext/dllexport5.C: Likewise.

Re: execute permissions in ada

2012-04-09 Thread Geert Bosch
On Apr 9, 2012, at 23:03, Mike Stump wrote: > I'd like to remove execute permissions for: > > gcc/ada/*.adb > > Ok? Sure. What about *.ads? -Geert

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-09 Thread Mike Frysinger
On Monday 09 April 2012 19:31:40 Adam Conrad wrote: > I realize that most people can't see past their own use case to understand > why a unique location for linkers is helpful, useful, and important for > some other people's use cases, but you either didn't read or chose to > ignore why using multi

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-09 Thread Mike Frysinger
On Thursday 05 April 2012 12:25:09 Konstantinos Margaritis wrote: > On Thu, 5 Apr 2012 11:55:14 -0400 Mike Frysinger wrote: > > note: i don't care about /lib/ld-linux-hf.so.3 or /lib/ld-linux.so.4 or > > /libhf/ld-linux.so.[34]. /lib// is really the only one i > > don't think doesn't belong. > >

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-09 Thread Jeff Law
On 04/09/2012 05:14 PM, Mike Frysinger wrote: tbh, i thought the ldso discussion was more "we've been talking about this for a long time, so let's just go with XXX" and then people moved on to the next topic (which was defining exactly what "hard float abi" meant wrt compiler flags). further, i

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-09 Thread Mike Frysinger
On Tuesday 10 April 2012 00:16:34 Jeff Law wrote: > On 04/09/2012 05:14 PM, Mike Frysinger wrote: > > tbh, i thought the ldso discussion was more "we've been talking about > > this for a long time, so let's just go with XXX" and then people moved > > on to the next topic (which was defining exactly

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-09 Thread Adam Conrad
On Tue, Apr 10, 2012 at 12:01:57AM -0400, Mike Frysinger wrote: > On Monday 09 April 2012 19:31:40 Adam Conrad wrote: > > I realize that most people can't see past their own use case to understand > > why a unique location for linkers is helpful, useful, and important for > > some other people's us

die CRs

2012-04-09 Thread Mike Stump
* g++.dg/cpp0x/udlit-raw-op.C: Don't use CRLF endings. * gcc.dg/tree-ssa/vrp59.c: Likewise. * gcc.dg/tree-ssa/vrp60.c: Likewise. Index: gcc.dg/tree-ssa/vrp60.c === --- gcc.dg/tree-ssa/vrp60.c (revision 186

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-09 Thread Paulo César Pereira de Andrade
Em 9 de abril de 2012 17:48, Adam Conrad escreveu: > On Thu, Apr 05, 2012 at 10:50:50AM +1200, Michael Hope wrote: >> On 4 April 2012 18:54, Jakub Jelinek wrote: >> > >> > If the agreement is that arm 32-bit softfp really needs to be installable >> > alongside 32-bit hardfp (and alongside aarch64

Ada testcase CR line endings

2012-04-09 Thread Mike Stump
So, I'd like to change all the ada testcases to use normal unix line endings. testsuite/gnat.dg/taft_type2_pkg.ads is an example if one such file, any objections?

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-09 Thread Jakub Jelinek
On Tue, Apr 10, 2012 at 05:17:36AM +, Adam Conrad wrote: > On Tue, Apr 10, 2012 at 12:01:57AM -0400, Mike Frysinger wrote: > > On Monday 09 April 2012 19:31:40 Adam Conrad wrote: > > > I realize that most people can't see past their own use case to understand > > > why a unique location for lin

Re: execute permissions in ada

2012-04-09 Thread Mike Stump
On Apr 9, 2012, at 8:39 PM, Geert Bosch wrote: > On Apr 9, 2012, at 23:03, Mike Stump wrote: >> I'd like to remove execute permissions for: >> >> gcc/ada/*.adb >> >> Ok? > > Sure. What about *.ads? I assume that was a friendly, please feel free to fix *.ads as well. Fixed: * a-assert

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-09 Thread Jeff Law
On 04/09/2012 11:17 PM, Adam Conrad wrote: Like I said, then, you didn't actually read or understand why proposing multilib paths doesn't work. You realize conceptually, I hope, that there's no guarantee of uniqueness in lib/lib64/lib32/libsf/libhf once you cross the base CPU architecture bound

Re: Ada testcase CR line endings

2012-04-09 Thread Arnaud Charlet
> So, I'd like to change all the ada testcases to use normal unix line endings. > > testsuite/gnat.dg/taft_type2_pkg.ads > > is an example if one such file, any objections? No objections.