Re: RFA: Add makefile for cross-configuration torture test

2011-04-14 Thread Joern Rennecke
Quoting Ralf Wildenhues ralf.wildenh...@gmx.de: Can you separate the configure and the build steps into separate make targets so that one can run one but not the other? Or would that have adverse effects on the load pattern (it might, due to the way make orders scheduling of jobs)? I've

Re: [v3] Re-implement is_constructible, add is_default_constructible, is_destructible, fix PR 48526

2011-04-14 Thread Magnus Fromreide
On Thu, 2011-04-14 at 00:48 +0200, Paolo Carlini wrote: Hi! over the last 2 days or so, Daniel Krugler - having filed in due course his Copyright Assignment - finally kindly contributed to the project shiny new std::is_constructible, std::is_default_constructible and

Re: [v3] Re-implement is_constructible, add is_default_constructible, is_destructible, fix PR 48526

2011-04-14 Thread Paolo Carlini
On 04/14/2011 10:32 AM, Magnus Fromreide wrote: This is just a nit, but why do you remove the /// is_function comments? Because was on the forward declaration, not on the implementation itself, where I wanted it. Paolo.

Re: [PATCH] add statistics counting to postreload, copy-rename, and math-opts

2011-04-14 Thread Richard Guenther
On Wed, Apr 13, 2011 at 8:43 PM, Nathan Froyd froy...@codesourcery.com wrote: On Wed, Apr 13, 2011 at 11:07:15AM +0200, Richard Guenther wrote: On Tue, Apr 12, 2011 at 5:09 PM, Nathan Froyd froy...@codesourcery.com wrote: Granted, but that fact should still be recorded.  The situation we

Re: [PATCH][RFC] Make called function type explicit, make function pointer type conversions useless

2011-04-14 Thread Eric Botcazou
This is the real fix for PR46076 that I wanted to persue. Make function pointer type conversions useless as to more aggressively be able to turn indirect into direct calls. This requires that we preserve the original type signature of the called function as presented by the frontend. The

Re: [PATCH][RFC] Make called function type explicit, make function pointer type conversions useless

2011-04-14 Thread Richard Guenther
On Thu, 14 Apr 2011, Eric Botcazou wrote: This is the real fix for PR46076 that I wanted to persue. Make function pointer type conversions useless as to more aggressively be able to turn indirect into direct calls. This requires that we preserve the original type signature of the called

Re: [Patch,testsuite,AVR]: Make PROGMEM variables const.

2011-04-14 Thread Denis Chertykov
2011/4/14 Georg-Johann Lay a...@gjlay.de: These test cases gcc.target/avr/trivial.c gcc.target/avr/torture/trivial.c put non-const variables in read-only section .progmem.data which causes an error. The patch rewrites the files to make respective variables real-only (and besides that fixes

Re: [PATCH PING] c++-specific bits of tree-slimming patches

2011-04-14 Thread Nathan Froyd
On Fri, Apr 08, 2011 at 01:50:24PM -0400, Jason Merrill wrote: On 03/24/2011 09:15 AM, Nathan Froyd wrote: + tree t = make_node (CASE_LABEL_EXPR); + + TREE_TYPE (t) = void_type_node; + SET_EXPR_LOCATION (t, input_location); As jsm and richi said, using input_location like this is a

[testsuite,AVR] test for PR46779, PR45291, PR41894

2011-04-14 Thread Georg-Johann Lay
This tests are intended to reveal the respective PRs because the test case is more stable under slight variations in code (both of application or compiler). This test case migh also be helpful for older versions of avr-gcc, in particular if PR41894 is actually fixed. 2011-04-14 Georg-Johann Lay

[PATCH] Handle more cases in get_pointer_modulus_and_residue

2011-04-14 Thread Richard Guenther
When using the C++ frontend to build gcc.dg/fold-bitand-4.c it currently fails to optimize because we don't handle a[i] properly in get_pointer_modulus_and_residue. We do have nice conservative code in get_object_alignment that just needs splitting out. Which is what the following patch does.

Re: Problem with ARM longcalls

2011-04-14 Thread Bernd Schmidt
Ping. Contains only changes outside config/arm. http://gcc.gnu.org/ml/gcc-patches/2011-03/msg01509.html Bernd On 03/23/2011 04:46 PM, Bernd Schmidt wrote: I've discovered a problem with -mlong-calls on ARM. The bug was first reported against a new target, but I'd copied the relevant code

Re: Problem with ARM longcalls

2011-04-14 Thread Richard Guenther
On Thu, Apr 14, 2011 at 2:40 PM, Bernd Schmidt ber...@codesourcery.com wrote: Ping.  Contains only changes outside config/arm. http://gcc.gnu.org/m/gcc-patches/2011-03/msg01509.html Ok. Thanks, Richard. Bernd On 03/23/2011 04:46 PM, Bernd Schmidt wrote: I've discovered a problem with

Re: Implement stack arrays even for unknown sizes

2011-04-14 Thread Dominique Dhumieres
I have opened PR48590 for at least one issue that I see. ... The fix for pr48590 (r172427) does not speedup fatigue.f90 compiled with -fstack-arrays. Dominique

[PATCH] Improve DSE a bit

2011-04-14 Thread Richard Guenther
Currently DSE does not allow for a call to be the redundant killing stmt. The following patch fixes that and also simplifies the DSE code by removing most of the domwalk stuff which wasn't necessary anymore. The alias-oracle is extended to handle some builtin calls as killing stmts so that we

Re: [PATCH][RFC] Make called function type explicit, make function pointer type conversions useless

2011-04-14 Thread Richard Guenther
On Thu, 14 Apr 2011, Eric Botcazou wrote: This is the real fix for PR46076 that I wanted to persue. Make function pointer type conversions useless as to more aggressively be able to turn indirect into direct calls. This requires that we preserve the original type signature of the called

Re: Try simplifying memory operands during optabs expansion

2011-04-14 Thread Richard Sandiford
Richard Henderson r...@redhat.com writes: On 03/31/2011 03:19 AM, Richard Sandiford wrote: * recog.h (insn_operand_data): Add an allows_mem field. * genoutput.c (output_operand_data): Initialize it. * optabs.c (maybe_legitimize_operand_same_code): New function.

Re: PATCH: PR middle-end/48608: Alignment adjust of local variables is lost

2011-04-14 Thread Richard Guenther
On Thu, Apr 14, 2011 at 3:34 PM, H.J. Lu hongjiu...@intel.com wrote: We have static unsigned int get_decl_align_unit (tree decl) {  unsigned int align = LOCAL_DECL_ALIGNMENT (decl);  return align / BITS_PER_UNIT; } LOCAL_DECL_ALIGNMENT may increase alignment for local variable.  But it

Re: PATCH: PR middle-end/48608: Alignment adjust of local variables is lost

2011-04-14 Thread H.J. Lu
On Thu, Apr 14, 2011 at 6:57 AM, Richard Guenther richard.guent...@gmail.com wrote: On Thu, Apr 14, 2011 at 3:34 PM, H.J. Lu hongjiu...@intel.com wrote: We have static unsigned int get_decl_align_unit (tree decl) {  unsigned int align = LOCAL_DECL_ALIGNMENT (decl);  return align /

Re: RFA: Add makefile for cross-configuration torture test

2011-04-14 Thread Jan-Benedict Glaw
On Thu, 2011-04-14 04:12:31 -0400, Joern Rennecke joern.renne...@embecosm.com wrote: --- contrib/config-list.mk(revision 0) +++ contrib/config-list.mk(revision 0) @@ -0,0 +1,98 @@ [...] +$(LIST): make-log-dir make-script-dir + -mkdir $@ + (cd $@ \ + ../../gcc/configure

Re: PATCH: PR middle-end/48608: Alignment adjust of local variables is lost

2011-04-14 Thread H.J. Lu
On Thu, Apr 14, 2011 at 7:09 AM, Richard Guenther richard.guent...@gmail.com wrote: On Thu, Apr 14, 2011 at 4:01 PM, H.J. Lu hjl.to...@gmail.com wrote: On Thu, Apr 14, 2011 at 6:57 AM, Richard Guenther richard.guent...@gmail.com wrote: On Thu, Apr 14, 2011 at 3:34 PM, H.J. Lu

Re: PATCH: PR middle-end/48608: Alignment adjust of local variables is lost

2011-04-14 Thread Richard Guenther
On Thu, Apr 14, 2011 at 4:27 PM, H.J. Lu hjl.to...@gmail.com wrote: On Thu, Apr 14, 2011 at 7:09 AM, Richard Guenther richard.guent...@gmail.com wrote: On Thu, Apr 14, 2011 at 4:01 PM, H.J. Lu hjl.to...@gmail.com wrote: On Thu, Apr 14, 2011 at 6:57 AM, Richard Guenther

Re: [PATCH 1/6] Linemap infrastructure for virtual locations

2011-04-14 Thread Tom Tromey
Dodji == Dodji Seketeli do...@redhat.com writes: Tom One thing to note is that there may be some code that assumes an Tom ordering of location values. If you hand out ordinary and macro Tom locations separately (which I think is what is going on), then code Tom doing this may break. [...]

C++ PATCH for c++/48531 (SFINAE and array of unknown bound)

2011-04-14 Thread Jason Merrill
Here's the one where we just need to check complain before giving an error. Tested x86_64-pc-linux-gnu, applying to trunk. commit a9fa1134b357ad03ac5279c926d67af75728062d Author: Jason Merrill ja...@redhat.com Date: Wed Apr 13 18:17:17 2011 -0400 PR c++/48531 * init.c

Minor C++ PATCH to fix lookup flags

2011-04-14 Thread Jason Merrill
When reviewing Rodrigo's patch for range-based for, I noticed that the flags for some calls to build_new_method_call were wrong. Fixed thus. Tested x86_64-pc-linux-gnu, applied to trunk. commit ea51123767c2512840f39002d2efac52111d1a7c Author: Jason Merrill ja...@redhat.com Date: Tue Apr 12

Re: Try simplifying memory operands during optabs expansion

2011-04-14 Thread Richard Henderson
On 04/14/2011 06:42 AM, Richard Sandiford wrote: The !side_effects_p check is the only change from last time. Tested on x86_64-linux-gnu, this time on a machine set up for building the 32-bit multilibs. Also tested on arm-linux-gnueabi. OK to install? Richard gcc/ * recog.h

Re: Remember/restore ALLOCA_FOR_VAR_P over tuples

2011-04-14 Thread Richard Guenther
On Thu, Apr 14, 2011 at 5:18 PM, Michael Matz m...@suse.de wrote: Hello, as my patch for stack-arrays in fortran reveals we a problem that VLA objects basically prevent all inlining to happen.  They're transformed into alloca calls and those are considered to disable inlining.  The (correct)

Re: [Patch, libfortran] PR 47571 Fix bootstrap failure on alpha*-dec-osf*

2011-04-14 Thread Steve Kargl
On Mon, Apr 11, 2011 at 11:18:53PM +0300, Janne Blomqvist wrote: the attached patch hopefully fixes the bootstrap failure on alpha*-dec-osf* due to said platform having clock_gettime() in librt, supporting weak symbols but not weak undefined symbols. The patch is the same as #38 in the PR

Re: Implement stack arrays even for unknown sizes

2011-04-14 Thread Michael Matz
Hi, On Thu, 14 Apr 2011, Michael Matz wrote: no stack-arrayswith stack-arrays no addtional options: 10.2s 8.8s + -fwhole-program: 7.1s 8.8s + -fwhole-program -flto: 10.1s

Re: Remember/restore ALLOCA_FOR_VAR_P over tuples

2011-04-14 Thread Eric Botcazou
We have a flag for this already on the CALL_EXPR. But we don't retain it over tuples, and hence we also don't look at it in inline_forbidden_p_stmt. This patch fixes both. (The strange testing of builtin-ness is because in CALL_EXPR the ALLOCA_FOR_VAR_P and CALL_FROM_THUNK_P flags are

Re: PATCH: PR middle-end/48608: Alignment adjust of local variables is lost

2011-04-14 Thread Richard Guenther
On Thu, Apr 14, 2011 at 5:48 PM, H.J. Lu hjl.to...@gmail.com wrote: On Thu, Apr 14, 2011 at 8:26 AM, Michael Matz m...@suse.de wrote: Hi, On Thu, 14 Apr 2011, Richard Guenther wrote: +  if (align DECL_ALIGN (decl)) +    DECL_ALIGN (decl) = align; Shouldn't this unconditionally set

Re: [PATCH][RFC] Make called function type explicit, make function pointer type conversions useless

2011-04-14 Thread Eric Botcazou
I'm working on a fix along this line, it needs some tweaks elsewhere. Indeed, here's what I needed to bootstrap on x86-64/Linux. * cfgexpand.c (expand_call_stmt): Rematerialize original function type. * config/i386/i386.c (ix86_expand_builtin): Use get_callee_fndecl. -- Eric

Re: [PATCH][RFC] Make called function type explicit, make function pointer type conversions useless

2011-04-14 Thread Richard Guenther
On Thu, 14 Apr 2011, Eric Botcazou wrote: I'm working on a fix along this line, it needs some tweaks elsewhere. Indeed, here's what I needed to bootstrap on x86-64/Linux. * cfgexpand.c (expand_call_stmt): Rematerialize original function type. * config/i386/i386.c

Re: [testsuite,AVR] test for PR46779, PR45291, PR41894

2011-04-14 Thread Denis Chertykov
2011/4/14 Georg-Johann Lay a...@gjlay.de: This tests are intended to reveal the respective PRs because the test case is more stable under slight variations in code (both of application or compiler). This test case migh also be helpful for older versions of avr-gcc, in particular if PR41894

Re: [PATCH][RFC] Make called function type explicit, make function pointer type conversions useless

2011-04-14 Thread Eric Botcazou
I have the following, the gimple_call_chain check is to prevent Ada bootstrap from failing. On x86? I suppose your patch is ok, maybe with simply not wrapping the NOP_EXPR around builtins. I can try that indeed... -- Eric Botcazou

[Patch,testsuite,avr]: add -finline-limit=0 to pr41885.c options

2011-04-14 Thread Georg-Johann Lay
This patchlet adds -finline-limit=0 to dg-options in testsuite/gcc.target/avr/torture/pr41885.c because otherwise optimizers will fold all tests and actually no test function is called when optimization is on. The test case still passes all tests. testsuite/ 2011-04-14 Georg-Johann Lay

Re: [PATCH,c++] fix PR objc++/48479, ICE in cxx_mark_addressable

2011-04-14 Thread Jason Merrill
OK. Jason

[build] Allow building libobjc_gc on Tru64 UNIX, Darwin

2011-04-14 Thread Rainer Orth
I recently tried to build with --enable-libobjc-gc. While this worked out of the box on Solaris, I ran intro trouble on both Tru64 UNIX and Darwin. On Tru64 UNIX, one needs to compile with -pthread since the boehm-gc headers include pthread.h which error out if -D_REENTRANT is missing. On

Re: [Patch,AVR]: FIX ICE in optabs due to bad rotate expander.

2011-04-14 Thread Georg-Johann Lay
Denis Chertykov schrieb: 2011/4/14 Georg-Johann Lay a...@gjlay.de: The rotlmode3 expanders (mode \in {HI,SI,DI}) violates synopsis by using 4 operands instead of 3. This runs in ICE in top of optabs.c:maybe_gen_insn. The right way to do this is to use match_dup, not match_operand. So the

[PATCH 8/n, i386]: Merge SSE and AVX patterns using enable attribute.

2011-04-14 Thread Uros Bizjak
Hello! Attached patch converts Intel SSE4.1 instructions and remaining sections. 2011-04-14 Uros Bizjak ubiz...@gmail.com * config/i386/sse.md (sse4_1): New mode attribute. (sse4_1_blendssemodesuffixavxmodesuffix): Macroize from avx_blendssemodesuffixavxmodesuffix and

[PATCH 8/n, i386]: Merge SSE and AVX patterns using enable attribute.

2011-04-14 Thread Uros Bizjak
Hello! Attached patch converts Intel SSE4.1 instructions and remaining sections. 2011-04-14 Uros Bizjak ubiz...@gmail.com * config/i386/sse.md (sse4_1): New mode attribute. (sse4_1_blendssemodesuffixavxmodesuffix): Macroize from avx_blendssemodesuffixavxmodesuffix and

Re: [Patch,testsuite,avr]: add -finline-limit=0 to pr41885.c options

2011-04-14 Thread Denis Chertykov
2011/4/14 Georg-Johann Lay a...@gjlay.de: This patchlet adds -finline-limit=0 to dg-options in testsuite/gcc.target/avr/torture/pr41885.c because otherwise optimizers will fold all tests and actually no test function is called when optimization is on. The test case still passes all tests.

Re: Convert legacy m68k options to .opt aliases

2011-04-14 Thread Gunther Nikl
Joseph S. Myers wrote: m68k-aout was obsoleted in 4.4 and removed in 4.5 - while some OSes with various odd object file formats are still supported, various newer features such as LTO may not work so well with them. I am aware that generic m68k-aout as target was deprecated and is gone now.

Re: [m68k] hookize FUNCTION_ARG co.

2011-04-14 Thread Gunther Nikl
Joseph S. Myers wrote: On Thu, 7 Apr 2011, Gunther Nikl wrote: Is there a (recommended) way how a subtarget can override hook functions? The implementation of the hook should contain if (preferred over #if) conditions on the particular ABI in use, which could be determined by a macro

RE: [build] Allow building libobjc_gc on Tru64 UNIX, Darwin

2011-04-14 Thread Nicola Pero
I recently tried to build with --enable-libobjc-gc. While this worked out of the box on Solaris, I ran intro trouble on both Tru64 UNIX and Darwin. [...] The following patch fixes both issues. [...] Ok for mainline if both pass? Yes. [and by the way, I think you're fixing PR

[PATCH] Fix unwind info with inline asm (PR middle-end/48597)

2011-04-14 Thread Jakub Jelinek
Hi! void foo (void) { register long s __asm (r13); register long t __asm (r14); register long u __asm (r15); asm volatile (xorq %%r12, %%r12 : : : r12); asm volatile (xorq %0, %0 : =r (s)); asm volatile (xorq %0, %0 : =r (t)); asm volatile (xorq %0, %0

Re: Implement stack arrays even for unknown sizes

2011-04-14 Thread Dominique Dhumieres
See http://gcc.gnu.org/ml/gcc-patches/2011-04/msg01087.html . ... With this patch on top of revision 172429 (and the second patch for -fstack-arrays) I now get Date Time : 14 Apr 2011 20:48:24 Test Name

Re: [build] Avoid ld -v error message with Sun ld on SPARC

2011-04-14 Thread Rainer Orth
David Miller da...@davemloft.net writes: Thanks Rainer, if you cons up a patch to add the .align directive I'll ACK that as well. That's my plan: I'll include such a patch in my S8 to S11 bootstraps over the weekend to make sure nothing breaks. Rainer --

Re: [lto/pph] Do not pack more bits than requested (issue4415044)

2011-04-14 Thread Jakub Jelinek
On Thu, Apr 14, 2011 at 02:50:53PM -0400, Diego Novillo wrote: @@ -1190,8 +1190,19 @@ bitpack_create (struct lto_output_stream *s) static inline void bp_pack_value (struct bitpack_d *bp, bitpack_word_t val, unsigned nbits) { - bitpack_word_t word = bp-word; + bitpack_word_t mask, word;

[pph] Rebuild compilation context from PPH images (2/6) (issue4413048)

2011-04-14 Thread Diego Novillo
When pickling type bitfields, we were not processing TYPE_ADDR_SPACE. * lto-streamer-in.c (unpack_ts_base_value_fields): Handle TYPE_ADDR_SPACE. * lto-streamer-out.c (pack_ts_base_value_fields): Likewise. diff --git a/gcc/ChangeLog.pph b/gcc/ChangeLog.pph index

[pph] Rebuild compilation context from PPH images (5/6) (issue4410045)

2011-04-14 Thread Diego Novillo
This patch factors out the allocation of sorted_fields_type into a new function so it can be called from the PPH reader. * class.c (sorted_fields_type_new): Factor out of ... (finish_struct_1): ... here. diff --git a/gcc/cp/class.c b/gcc/cp/class.c index 1325260..b040449

[doc] Remove references to mips-tfile on MIPS

2011-04-14 Thread Rainer Orth
I noticed that there were a couple of references to mips-tfile on MIPS, but this is only used on Tru64 UNIX these days. The following patch corrects this. Included in an i386-pc-solaris2.11 bootstrap to ensure the doc changes are syntactically correct. The MIPS changes should be obvious, though

Re: [doc] Remove references to mips-tfile on MIPS

2011-04-14 Thread Richard Sandiford
Rainer Orth r...@cebitec.uni-bielefeld.de writes: Ok for mainline if that passes? Definitely, thanks. Richard 2011-04-08 Rainer Orth r...@cebitec.uni-bielefeld.de * config/mips/mips.h (DBX_OUTPUT_SOURCE_LINE): Remove. * config/mips/mips.opt (mmips-tfile): Remove. *

Re: [PATCH][RFC] Make called function type explicit, make function pointer type conversions useless

2011-04-14 Thread Eric Botcazou
I suppose your patch is ok, maybe with simply not wrapping the NOP_EXPR around builtins. Here's what I've installed after bootstrapping/regtesting on x86_64-suse-linux. 2011-04-14 Eric Botcazou ebotca...@adacore.com * cfgexpand.c (expand_call_stmt): Rematerialize the original

[pph] Macro Validation Correction (issue4425041)

2011-04-14 Thread Lawrence Crowl
An earlier change reduced the number of bits in cpp_hashnode.directive_index from 7 to 5, as that was sufficient for indexing the directives. Tom Tromey asked for a static check on the size. This patch adds that check. Unfortunately, five bits are not sufficient for the alternate use of

Ping Re: [patch, ARM] Fix PR48325, support POST_INC/PRE_DEC for NEON struct modes

2011-04-14 Thread Chung-Lin Tang
Ping. On 2011/3/31 10:57 PM, Chung-Lin Tang wrote: This PR doesn't exactly trigger currently on trunk; a REG_DEAD note that occurs in trunk, but not in the 4.5-based compilers which this bug was reported for, currently blocks auto-inc-dec from doing its job, and just happens to avoid this

Go patch committed: Use the backend interface for select statements

2011-04-14 Thread Ian Lance Taylor
This patch to the Go frontend uses the backend interface for select statements. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian 2011-04-14 Ian Lance Taylor i...@google.com * go-gcc.cc (Backend::error_statement): New function. Index:

Re: [Patch, libfortran] PR 47571 Fix bootstrap failure on alpha*-dec-osf*

2011-04-14 Thread Janne Blomqvist
On Thu, Apr 14, 2011 at 18:27, Steve Kargl s...@troutmask.apl.washington.edu wrote: On Mon, Apr 11, 2011 at 11:18:53PM +0300, Janne Blomqvist wrote: the attached patch hopefully fixes the bootstrap failure on alpha*-dec-osf* due to said platform having clock_gettime() in librt, supporting

Re: PATCH [trunk] gengtype should generate ggc_alloc macros in plugin mode on for plugin files

2011-04-14 Thread Basile Starynkevitch
On Fri, 15 Apr 2011 06:38:46 +0300 Laurynas Biveinis laurynas.bivei...@gmail.com wrote: 2011/4/11 Basile Starynkevitch bas...@starynkevitch.net: The attached file improves the situation, by flagging input files and generating ggc_alloc macros only relevant to plugin files. What about