Re: [PR48866] three alternative fixes

2012-04-09 Thread Alexandre Oliva
On Jun 2, 2011, Alexandre Oliva aol...@redhat.com wrote: On May 30, 2011, Alexandre Oliva aol...@redhat.com wrote: On May 30, 2011, Alexandre Oliva aol...@redhat.com wrote: I have 3 different, mutually exclusive patches that fix PR 48866. The problem is exponential time while dealing with

Re: fix left-over debug insns in DCE

2012-04-09 Thread Alexandre Oliva
On Jun 6, 2011, Alexandre Oliva aol...@redhat.com wrote: On Jun 6, 2011, Eric Botcazou ebotca...@adacore.com wrote: It might be too late for DF to do anything sensible to preserve the debug info rather than just throw it away, as your partial approval suggests. OK, let me think about this

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

2012-04-09 Thread Alexandre Oliva
This patch is a small improvement to DF, for an issue I noticed while investigating PR 50826. We take note of, and may emit debug temps for, REGs for which we won't emit REG_DEAD/REG_UNUSED notes. This is a waste, removed with this patch. Regstrapped on x86_64-linux-gnu and i686-linux-gnu. Ok

Re: [PATCH] Fix ICE in expand_cse_reciprocals (PR tree-optimization/42078)

2012-04-09 Thread Alexandre Oliva
On Jun 3, 2011, Alexandre Oliva aol...@redhat.com wrote: According to http://gcc.gnu.org/ml/gcc-patches/2009-11/msg01082.html on Nov 20, 2009, Alexandre Oliva aol...@redhat.com wrote: On Nov 19, 2009, Richard Guenther richard.guent...@gmail.com wrote: In fact this exchanging of the LHS (or

Re: fix incorrect debug temp added by df-problems

2012-04-09 Thread Alexandre Oliva
On Oct 27, 2011, Alexandre Oliva aol...@redhat.com wrote: pr43165.c -Os -g exercises a bug in df-problems WRT the introduction of debug temps. After pro_and_epilogue, the function was optimized to this: push bp mov bp,sp # debug insn referencing bp pop bp I don't know why we bother to

Re: don't force debug insns after their PREV_INSNs

2012-04-09 Thread Alexandre Oliva
On Jun 3, 2011, Bernd Schmidt ber...@codesourcery.com wrote: On 06/03/2011 04:47 PM, Alexandre Oliva wrote: https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=677681 can be “fixed” by disabling the artificial dependency of a debug insn on its previous insn. Debug insns will often depend

Re: debug insns in SMS

2012-04-09 Thread Alexandre Oliva
On May 4, 2011, Revital1 Eres e...@il.ibm.com wrote: Hello Alexandre I think this will restore proper functioning to SMS in the presence of debug insns. A while ago, we'd never generate deps of non-debug insns on debug insns. I introduced them to enable sched to adjust (reset) debug insns

[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

[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

[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

[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

[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?

Re: [PATCH] Fix PR52614

2012-04-09 Thread Richard Guenther
On Fri, Apr 6, 2012 at 12:15 AM, Ian Lance Taylor i...@google.com wrote: Richard Guenther richard.guent...@gmail.com writes: On Thu, Apr 5, 2012 at 6:22 AM, Mike Stump mikest...@comcast.net wrote: On Apr 4, 2012, at 7:56 PM, William J. Schmidt wrote: There seems to be tacit agreement that the

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 lopeziba...@gmail.com 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

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

2012-04-09 Thread Richard Guenther
On Fri, Apr 6, 2012 at 10:13 AM, Richard Sandiford rdsandif...@googlemail.com wrote: Richard Guenther richard.guent...@gmail.com 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

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 aol...@redhat.com PR debug/51570 * var-tracking.c (expand_depth): New type. (onepart_aux, expand_loc_callback_data): Change depth type to it.

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 d...@google.com writes: Doug On Thu, Mar 15, 2012 at 11:54 AM, Mike Stump mikest...@comcast.net 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.,

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 tro...@redhat.com wrote: Doug == Doug Evans d...@google.com writes: Doug On Thu, Mar 15, 2012 at 11:54 AM, Mike Stump mikest...@comcast.net wroteDoug btw, is there any interest in mirroring gcc/contrib - src/contrib? Doug [instead of gdb having its

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 came

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 the

[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

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 ger...@pfeifer.com wrote: On Thu, 29 Mar 2012, Sriraman Tallam wrote: Hi,     I have made a new patch to only have two builtins : * __builtin_cpu_is (CPUNAME) * __builtin_cpu_supports (FEATURE) apart from the cpu init builtin,

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 lopeziba...@gmail.com wrote: On 9 April 2012 12:43, Gabriel Dos Reis g...@integrable-solutions.net wrote: On Mon, Apr 9, 2012 at 5:12 AM, Manuel López-Ibáñez lopeziba...@gmail.com wrote:        * c-common.h (c_common_initialize_diagnostics):

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 ger...@pfeifer.com 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

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

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 fdum...@gcc.gnu.org PR libstdc++/52476 * include/bits/hashtable.h (_Hashtable::_M_rehash_aux): Add. (_Hashtable::_M_rehash): Use the latter. *

Re: [PATCH] Caret diagnostics

2012-04-09 Thread Manuel López-Ibáñez
): New. (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

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 ja...@redhat.com 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

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 era...@google.com wrote: On Sun, Mar 25, 2012 at 12:22 PM, Jan Hubicka hubi...@ucw.cz 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?

[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

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

2012-04-09 Thread Kaz Kojima
Oleg Endo oleg.e...@t-online.de 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

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), then

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/tuple/ldso 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,

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

[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,

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 multilib

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/triplet/ldso 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, it

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 what

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 use

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

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 adcon...@debian.org escreveu: On Thu, Apr 05, 2012 at 10:50:50AM +1200, Michael Hope wrote: On 4 April 2012 18:54, Jakub Jelinek ja...@redhat.com wrote: If the agreement is that arm 32-bit softfp really needs to be installable alongside 32-bit

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 linkers is

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.ads: Remove