Re: [PATCH] Don't bypass blocks with multiple latch edges (PR middle-end/54838)

2012-11-29 Thread Richard Biener
On Wed, Nov 28, 2012 at 7:24 PM, Marek Polacek pola...@redhat.com wrote: On Wed, Nov 28, 2012 at 10:52:17AM +0100, Eric Botcazou wrote: No, I don't think that's the problem. The above messages are admittedly a bit terse, they should say: JUMP-BYPASS: Proved reg 59 in jump_insn 15 equals

Re: patch to fix PR55512

2012-11-29 Thread Jakub Jelinek
On Wed, Nov 28, 2012 at 12:43:15PM -0500, Vladimir Makarov wrote: The following patch fixes http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55512 The patch was successfully tested and bootstrapped on x86/x86-64. Committed as rev. 193901. 2012-11-28 Vladimir Makarov

Re: [PATCH] Support -fuse-ld=bfd and -fuse-ld=gold

2012-11-29 Thread Richard Biener
On Thu, Nov 29, 2012 at 5:18 AM, H.J. Lu hjl.to...@gmail.com wrote: From: H.J. Lu hjl.to...@gmail.com To: gcc-patches@gcc.gnu.org Cc: Joseph Myers jos...@codesourcery.com, Paolo Bonzini bonz...@gnu.org Bcc: Subject: [PATCH] Support -fuse-ld=bfd and -fuse-ld=gold Reply-To: Hi, Binutils

Re: [PATCH] Don't bypass blocks with multiple latch edges (PR middle-end/54838)

2012-11-29 Thread Steven Bosscher
On Thu, Nov 29, 2012 at 9:34 AM, Richard Biener wrote: On Wed, Nov 28, 2012 at 7:24 PM, Marek Polacek wrote: I admit I'm not sure what to look at, maybe cprop should have in properties_destroyed PROP_loops? Well, then we need to remove one assert in loop-init.c. So something like:

Re: [PATCH, generic] Fix for define_subst

2012-11-29 Thread Michael Zolotukhin
So, ok for commit this patch? Changelog: 2012-11-29 Michael Zolotukhin michael.v.zolotuk...@intel.com * gensupport.c (maybe_eval_c_test): Remove not-null check for expr. * read-rtl.c (apply_iterators): Initialize condition with instead of NULL. On 28 November 2012

Re: [Patch, Fortran, committed] PR52161 - Fix bound checking of SYNC IMAGES( n )

2012-11-29 Thread Tobias Burnus
Tobias Burnus wrote: Committed as Rev. 193908 Looking at the code again, the condition for -fcoarray=lib was wrong and never triggered (-fcoarray=single was okay). Fixed by the following commit after building and regtesting. Tobias Index: gcc/fortran/ChangeLog

Re: [PATCH] Support -fuse-ld=bfd and -fuse-ld=gold

2012-11-29 Thread Markus Trippelsdorf
On 2012.11.29 at 09:43 +0100, Richard Biener wrote: On Thu, Nov 29, 2012 at 5:18 AM, H.J. Lu hjl.to...@gmail.com wrote: From: H.J. Lu hjl.to...@gmail.com To: gcc-patches@gcc.gnu.org Cc: Joseph Myers jos...@codesourcery.com, Paolo Bonzini bonz...@gnu.org Bcc: Subject: [PATCH] Support

Re: [PATCH] Don't bypass blocks with multiple latch edges (PR middle-end/54838)

2012-11-29 Thread Richard Biener
On Thu, Nov 29, 2012 at 9:56 AM, Steven Bosscher stevenb@gmail.com wrote: On Thu, Nov 29, 2012 at 9:34 AM, Richard Biener wrote: On Wed, Nov 28, 2012 at 7:24 PM, Marek Polacek wrote: I admit I'm not sure what to look at, maybe cprop should have in properties_destroyed PROP_loops? Well,

[PATCH,ARM] Subdivide alu into alu_reg and simple_alu_imm

2012-11-29 Thread Greta Yorsh
For attribute named type, subdivide alu into alu_reg and simple_alu_imm. Set type attribute as appropriate in define_insn patterns with immediate operands. Update pipeline descriptions to use the new values of type attribute. No regression on qemu arm-none-eabi -mcpu=cortex-a15/cortex-a7.

Re: [PATCH, RFC] PR 55415 : Pessimistic misalignment from eipa_sra pass

2012-11-29 Thread Martin Jambor
Hi, thanks for the review. When writing a reply I realized I indeed made a mistake or two in the part concerning prev_base and the code was not what it intended to be. I'll re-write it today. Nevertheless, I also have a question regarding a different place of the patch: On Wed, Nov 28, 2012

[patch mingw]: Correct page-size granularity and retry for address-mapping on mingw-hosts.

2012-11-29 Thread Kai Tietz
Hi, this patch fixes a latent issue about page-size granularity for Windows OSes - like Vista, Win7, etc. - and avoid a race happening for mapping of memory for multiple instances of compiler. ChangeLog 2012-11-29 Kai Tietz * host-mingw32.c (va_granularity): Make none-const.

Re: [PATCH,ARM] Subdivide alu into alu_reg and simple_alu_imm

2012-11-29 Thread Richard Earnshaw
; ??? Check Thumb-2 split length (define_insn_and_split *arm_subsi3_insn - [(set (match_operand:SI 0 s_register_operand =r,r,rk,r) - (minus:SI (match_operand:SI 1 reg_or_int_operand rI,r,k,?n) - (match_operand:SI 2 reg_or_int_operand r,rI,r, r)))] + [(set

[patch mingw]: Cleanup about static/shared libgcc

2012-11-29 Thread Kai Tietz
Hi, this patch synchronize handling - and fixes latent bug - of static/shared flags with cygwin-host. in static case prior code added to linker-script reference to shared libgcc, which is obviously wrong. Due link-order the shared version doesn't get actual linked, but nevertheless there is a

Re: PR55124 - tentative patch for ICE in PRE

2012-11-29 Thread Richard Biener
On Wed, Nov 28, 2012 at 3:05 PM, Tom de Vries vr...@codesourcery.com wrote: On 27/11/12 13:41, Richard Biener wrote: On Mon, Nov 19, 2012 at 3:33 AM, Tom de Vries tom_devr...@mentor.com wrote: Richard, Consider the PR55124 example test.c: ... int a, b; long c; static void inline f2

Re: [0/8] Add optabs alternatives for insv, extv and extzv

2012-11-29 Thread Richard Sandiford
Eric Botcazou ebotca...@adacore.com writes: This is because the structure we are given is: (mem/v/j:SI (reg/v/f:SI 110 [ t ]) [3 t_2(D)-a+0 S1 A32]) i.e. a 1-byte SImode reference. The strange size comes from the set_mem_attributes_minus_bitpos code that was mentioned earlier in the

Re: PR fortran/51727: make module files reproducible, question on C++ in gcc

2012-11-29 Thread Tobias Schlüter
Dear gfortraners, On 2012-10-14 23:35, Janne Blomqvist wrote: - I'd be vary wrt backporting, in my experience the module.c code is somewhat fragile and easily causes regressions. In any case, AFAICS PR 51727 is not a regression. Can this be reconsidered, as the benefits for users seem to be

Re: [PATCH] Instance information in DWARF discriminators

2012-11-29 Thread Thomas Quinot
* Richard Biener, 2012-11-28 : You need to stream the id with LTO, no? Also increasing the size of line_map_ordinary might not be the most welcom change ... Thanks for your feedback Richard, I'll take care of adding the missing LTO pieces, and improve documentation of the flag. As for the

Re: [Patch, Fortran] PR55469 - fix I/O memory leaks in case of failure and iostat= being present

2012-11-29 Thread Tobias Burnus
Tobias Burnus wrote: l_push_char allocates memory which is freed with free_line. However, currently, the memory is not always freed when calling generate_error. If one aborts, that's fine. However, generate_error can also set the iostat variable. Updated version: Corrected PR number - and

Re: [PATCH, RFC] PR 55415 : Pessimistic misalignment from eipa_sra pass

2012-11-29 Thread Richard Biener
On Thu, Nov 29, 2012 at 11:06 AM, Martin Jambor mjam...@suse.cz wrote: Hi, thanks for the review. When writing a reply I realized I indeed made a mistake or two in the part concerning prev_base and the code was not what it intended to be. I'll re-write it today. Nevertheless, I also have

Re: [Patch, Fortran] PR55469 - fix I/O memory leaks in case of failure and iostat= being present

2012-11-29 Thread Janne Blomqvist
On Thu, Nov 29, 2012 at 1:38 PM, Tobias Burnus bur...@net-b.de wrote: Tobias Burnus wrote: l_push_char allocates memory which is freed with free_line. However, currently, the memory is not always freed when calling generate_error. If one aborts, that's fine. However, generate_error can also

[patch c++]: 1 of 7 Fix for PR target/53912 bootstrap fails using default c++ mode in stage 2 and 3 for native x86_64-w64-mingw32

2012-11-29 Thread Kai Tietz
Hello, this trivial patch fixes a bootstrap issue on LLP64 hosts. cp/ PR target/53912 * class.c (dump_class_hierarchy_r): Cast from pointer via uintptr_t. (dump_vtable): Likewise. Tested for i686-w64-mingw32, x86_64-w64-mingw32, and

[patch]: 2 of 7 Fix of PR target/53912 bootstrap fails using default c++ mode in stage 2 and 3 for native x86_64-w64-mingw32

2012-11-29 Thread Kai Tietz
Hello, this trivial patch fixes a bootstrap issue on LLP64 hosts. ChangeLog 2012-11-29 Kai Tietz PR target/53912 * ggc-common.c (POINTER_HASH): Cast from pointer via intptr_t. Tested for i686-w64-mingw32, x86_64-w64-mingw32, and x86_64-unknown-gnu-linux. Ok for apply?

[patch]: 3 of 7 Fix of PR target/53912 bootstrap fails using default c++ mode in stage 2 and 3 for native x86_64-w64-mingw32

2012-11-29 Thread Kai Tietz
Hello, this trivial patch fixes a bootstrap issue on LLP64 hosts. ChangeLog 2012-11-29 Kai Tietz PR target/53912 * pointer-set.c (hash1): Cast from pointer via uintptr_t. Tested for i686-w64-mingw32, x86_64-w64-mingw32, and x86_64-unknown-gnu-linux. Ok for apply? Regards, Kai

[patch prefix.c]: 4 of 7 Fix of PR target/53912 bootstrap fails using default c++ mode in stage 2 and 3 for native x86_64-w64-mingw32

2012-11-29 Thread Kai Tietz
Hello, this trivial patch fixes a bootstrap issue on LLP64 hosts. ChangeLog 2012-11-29 Kai Tietz PR target/53912 * prefix.c (lookup_key): Explicit cast return-type of xmalloc/xrealloc to char *. Tested for i686-w64-mingw32, x86_64-w64-mingw32, and

[patch print-tree.c]: 5 of 7 Fix of PR target/53912 bootstrap fails using default c++ mode in stage 2 and 3 for native x86_64-w64-mingw32

2012-11-29 Thread Kai Tietz
Hello, this trivial patch fixes a bootstrap issue on LLP64 hosts. ChangeLog 2012-11-29 Kai Tietz PR target/53912 * print-tree.c (print_node): Cast from pointer via uintptr_t. Tested for i686-w64-mingw32, x86_64-w64-mingw32, and x86_64-unknown-gnu-linux. Ok for apply? Regards,

[patch stmt.c]: 6 of 7 Fix of PR target/53912 bootstrap fails using default c++ mode in stage 2 and 3 for native x86_64-w64-mingw32

2012-11-29 Thread Kai Tietz
Hello, this trivial patch fixes a bootstrap issue on LLP64 hosts. ChangeLog 2012-11-29 Kai Tietz PR target/53912 * stmt.c (compute_cases_per_edge): Cast from pointer via intptr_t. (expand_case): Likewise. Tested for i686-w64-mingw32, x86_64-w64-mingw32, and

[patch tree-dump.c]: 7 of 7 Fix of PR target/53912 bootstrap fails using default c++ mode in stage 2 and 3 for native x86_64-w64-mingw32

2012-11-29 Thread Kai Tietz
Hello, this trivial patch fixes a bootstrap issue on LLP64 hosts. ChangeLog 2012-11-29 Kai Tietz PR target/53912 * tree-dump.c (dump_pointer): Cast from pointer via intptr_t. Tested for i686-w64-mingw32, x86_64-w64-mingw32, and x86_64-unknown-gnu-linux. Ok for apply? Regards,

Re: PR55124 - tentative patch for ICE in PRE

2012-11-29 Thread Tom de Vries
On 29/11/12 11:26, Richard Biener wrote: I'm continuing trying to move value-ids back to PRE land. Your patch would be nice in a form that verifies the order is indeed topological, maybe you can re-work it in a way that does this in sorted_array_from_bitmap_set in a ENABLE_CHECKING piece?

[tsan] instrument_expr improvements

2012-11-29 Thread Jakub Jelinek
Hi! As discussed earlier on mailing list and on IRC with Richard, pt_solution_includes can be used for testing of whether var might escape current function (for -O0 unfortunately returns true even for !may_be_aliased automatic vars, so I'm testing for that too), and there are IMHO tons of

Re: [patch c++]: 1 of 7 Fix for PR target/53912 bootstrap fails using default c++ mode in stage 2 and 3 for native x86_64-w64-mingw32

2012-11-29 Thread Pedro Alves
On 11/29/2012 12:03 PM, Kai Tietz wrote: Hello, this trivial patch fixes a bootstrap issue on LLP64 hosts. But it's trivial at the expense of truncating the pointer. Looking around, I'd think that something like casting to hwi (after uintptr_t), and then use HOST_WIDE_INT_PRINT_HEX as format

Re: [PATCH, RFC] PR 55415 : Pessimistic misalignment from eipa_sra pass

2012-11-29 Thread Martin Jambor
Hi, On Thu, Nov 29, 2012 at 12:54:20PM +0100, Richard Biener wrote: On Thu, Nov 29, 2012 at 11:06 AM, Martin Jambor mjam...@suse.cz wrote: Hi, thanks for the review. When writing a reply I realized I indeed made a mistake or two in the part concerning prev_base and the code was not

Re: [patch c++]: 1 of 7 Fix for PR target/53912 bootstrap fails using default c++ mode in stage 2 and 3 for native x86_64-w64-mingw32

2012-11-29 Thread Kai Tietz
2012/11/29 Pedro Alves pal...@redhat.com: On 11/29/2012 12:03 PM, Kai Tietz wrote: Hello, this trivial patch fixes a bootstrap issue on LLP64 hosts. But it's trivial at the expense of truncating the pointer. Looking around, I'd think that something like casting to hwi (after uintptr_t),

Re: [patch c++]: 1 of 7 Fix for PR target/53912 bootstrap fails using default c++ mode in stage 2 and 3 for native x86_64-w64-mingw32

2012-11-29 Thread Pedro Alves
On 11/29/2012 12:45 PM, Kai Tietz wrote: 2012/11/29 Pedro Alves pal...@redhat.com: On 11/29/2012 12:03 PM, Kai Tietz wrote: Hello, this trivial patch fixes a bootstrap issue on LLP64 hosts. But it's trivial at the expense of truncating the pointer. Looking around, I'd think that something

Re: PR55124 - tentative patch for ICE in PRE

2012-11-29 Thread Richard Biener
On Thu, Nov 29, 2012 at 1:14 PM, Tom de Vries tom_devr...@mentor.com wrote: On 29/11/12 11:26, Richard Biener wrote: I'm continuing trying to move value-ids back to PRE land. Your patch would be nice in a form that verifies the order is indeed topological, maybe you can re-work it in a way

constexpr vector operations

2012-11-29 Thread Marc Glisse
Hello, this patch fixes several ICEs when using constexpr SIMD vectors. Support for subscripting is still missing though, which is why I am not adding some static_asserts to the testcase. I don't use build_vector_from_ctor because it doesn't check for non-constant elements and doesn't handle

Re: [patch tree-dump.c]: 7 of 7 Fix of PR target/53912 bootstrap fails using default c++ mode in stage 2 and 3 for native x86_64-w64-mingw32

2012-11-29 Thread Kai Tietz
Updated variant using HOST_WIDE_INT_PRINT. Tested for i686-w64-mingw32 and x86_64-w64-mingw32. Ok for apply? Kai Index: tree-dump.c === --- tree-dump.c (Revision 193925) +++ tree-dump.c (Arbeitskopie) @@ -177,7 +177,8 @@ void

[PATCH] PRE TLC

2012-11-29 Thread Richard Biener
When working on PR55124 I noticed we do some useless work in PRE. This cleans it up and makes it more obvious what we do in compute_avail. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2012-11-29 Richard Biener rguent...@suse.de * tree-ssa-pre.c

Re: [PATCH] Support -fuse-ld=bfd and -fuse-ld=gold

2012-11-29 Thread H.J. Lu
On Thu, Nov 29, 2012 at 1:10 AM, Markus Trippelsdorf mar...@trippelsdorf.de wrote: On 2012.11.29 at 09:43 +0100, Richard Biener wrote: On Thu, Nov 29, 2012 at 5:18 AM, H.J. Lu hjl.to...@gmail.com wrote: From: H.J. Lu hjl.to...@gmail.com To: gcc-patches@gcc.gnu.org Cc: Joseph Myers

Re: [tsan] instrument_expr improvements

2012-11-29 Thread Dmitry Vyukov
On Thu, Nov 29, 2012 at 4:24 PM, Jakub Jelinek ja...@redhat.com wrote: Hi! As discussed earlier on mailing list and on IRC with Richard, pt_solution_includes can be used for testing of whether var might escape current function (for -O0 unfortunately returns true even for !may_be_aliased

Re: [tsan] instrument_expr improvements

2012-11-29 Thread Richard Biener
On Thu, 29 Nov 2012, Jakub Jelinek wrote: Hi! As discussed earlier on mailing list and on IRC with Richard, pt_solution_includes can be used for testing of whether var might escape current function (for -O0 unfortunately returns true even for !may_be_aliased automatic vars, so I'm testing

[0/3][ARM] AArch32 NEON vrint builtins and intrinsics

2012-11-29 Thread Kyrylo Tkachov
Hi all, This set of patches adds support for the AArch32 vrint NEON builtins. It also adds related NEON intrinsics. It is organised in 3 patches: * The first patch adds support for the builtin versions of the instructions. * The second is a small testsuite patch that adds the effective target

[PATCH][ARM][2/3] AArch32 NEON vrint builtins and intrinsics

2012-11-29 Thread Kyrylo Tkachov
Hi all, This patch adds an effective target check to the testsuite for ARMv8 NEON support. A corresponding add_options procedure is added. This is used by the AArch32 NEON intrinsics tests. An entry in the documentation is added as well. Ok for trunk? Thanks, Kyrill gcc/ChangeLog 2012-11-29

[PATCH][ARM][3/3] AArch32 NEON vrint builtins and intrinsics

2012-11-29 Thread Kyrylo Tkachov
Hi all, This patch adds the intrinsics support for the vrnd intrinsics that are implemented by the vrint instructions. The .ml scripts contain the new information and should used to regenerate the arm_neon.h header file, tests and documentation. In particular: * config/arm/arm_neon.h should be

Re: [patch] Fix PR middle-end/55321

2012-11-29 Thread Eric Botcazou
Well, there's a call at the tree level for __sync_synchronize as well. So the question is: why does __s_s ICE but memcmp doesn't? I suppose it's the mere rarity with which HAVE_cmpmemsi is defined, and expansion of gen_cmpmemsi fails. There are plenty of other calls to

Re: [PATCH, RFC] PR 55415 : Pessimistic misalignment from eipa_sra pass

2012-11-29 Thread Richard Biener
On Thu, Nov 29, 2012 at 1:33 PM, Martin Jambor mjam...@suse.cz wrote: Hi, On Thu, Nov 29, 2012 at 12:54:20PM +0100, Richard Biener wrote: On Thu, Nov 29, 2012 at 11:06 AM, Martin Jambor mjam...@suse.cz wrote: Hi, thanks for the review. When writing a reply I realized I indeed made a

Re: [RFA/ARM] Fix PR54974: Thumb literal pools don't handle PC rounding

2012-11-29 Thread Matthew Gretton-Dann
On 24 November 2012 00:27, Ramana Radhakrishnan ramana@googlemail.com wrote: On Wed, Nov 21, 2012 at 7:59 PM, Matthew Gretton-Dann matthew.gretton-d...@linaro.org wrote: [snip] The fix is to decrease the pool_range of all insns by 2 when generating Thumb code. There is no need to change

Re: [RFA/4.7/ARM] Backport arm-*-linux-gnueabihf triplet support to 4.7

2012-11-29 Thread Matthew Gretton-Dann
PING On 20 November 2012 20:34, Matthew Gretton-Dann matthew.gretton-d...@linaro.org wrote: All, This patch backports Matthais Klose's arm*-*-linux-gnueabihf triplet support patch of 2012-10-15 to 4.7. The backport was not clean as 4.8 has obsoleted various arm*-*-* triplets which are

[patch] RFA: more fixes for PR55006

2012-11-29 Thread Steven Bosscher
On Thu, Nov 29, 2012 at 12:53 AM, Steven Bosscher wrote: Attached is a different fix. It splits DF_REF_IN_NOTE in two: One flag for each kind of note. This allows the dead note removal code to distinguish the source note for the EQ_USES. I needed to remove one flag to keep the df_ref_flags

RE: [PATCH,ARM] Subdivide alu into alu_reg and simple_alu_imm

2012-11-29 Thread Greta Yorsh
-Original Message- From: Richard Earnshaw Sent: 29 November 2012 10:12 To: Greta Yorsh Cc: GCC Patches; Ramana Radhakrishnan; ni...@redhat.com; p...@codesourcery.com Subject: Re: [PATCH,ARM] Subdivide alu into alu_reg and simple_alu_imm ; ??? Check Thumb-2 split length

Re: [PATCH][AARCH64] Refactor constant generation

2012-11-29 Thread Marcus Shawcroft
On 26/11/12 16:50, Sofiane Naci wrote: Hi, Constant building in the AArch64 backend spits out assembly code, which affects scheduling of the generated code. This patch rewrites the code to use RTL patterns. A full aarch64-none-elf regression run shows no issues. Thanks Sofiane -

Re: [PATCH,ARM] Subdivide alu into alu_reg and simple_alu_imm

2012-11-29 Thread Richard Earnshaw
On 29/11/12 14:58, Greta Yorsh wrote: -Original Message- From: Richard Earnshaw Sent: 29 November 2012 10:12 To: Greta Yorsh Cc: GCC Patches; Ramana Radhakrishnan; ni...@redhat.com; p...@codesourcery.com Subject: Re: [PATCH,ARM] Subdivide alu into alu_reg and simple_alu_imm ; ???

Re: [Patch AArch64-4.7] Backport - Refactor Advanced SIMD builtin initialisation.

2012-11-29 Thread Marcus Shawcroft
On 28/11/12 11:07, James Greenhalgh wrote: Hi, The backport for this patch was not entirely clean, __builtin_thread_pointer was made a front end builtin on the 4.8 branch, but remains a back-end builtin on aarch64-4.7. Otherwise, there are no differences between this patch and the patch which

Re: [PATCH] Instance information in DWARF discriminators

2012-11-29 Thread Tom Tromey
Thomas == Thomas Quinot qui...@adacore.com writes: Thomas As for the size impact on line_map_ordinary, indeed that's an additional Thomas int here, but I'm not sure we allocate so many objects of that kind that Thomas the increase is significant. Could you please measure it? My guess is that it

Re: constexpr vector operations

2012-11-29 Thread Jason Merrill
OK. Jason

Re: [0/8] Add optabs alternatives for insv, extv and extzv

2012-11-29 Thread Eric Botcazou
I agree that this kind of MEM is less than ideal, but I thought: set_mem_attributes_minus_bitpos (to_rtx, to, 0, bitpos); said that the attributes of TO_RTX will to be TO _once a pending offset-and- narrowing operation has been applied_. So we have: /* If we modified

Re: [PATCH] Don't bypass blocks with multiple latch edges (PR middle-end/54838)

2012-11-29 Thread Marek Polacek
On Thu, Nov 29, 2012 at 09:34:31AM +0100, Richard Biener wrote: Definitely not - that means to not preserve loops until after cprop. The goal is to preserve loops everywhere! Yikes, sorry, it wasn't clear to me what PROP_loops really does. Anyway, I think I have a better fix now. The problem

Re: [PATCH] Support -fuse-ld=bfd and -fuse-ld=gold

2012-11-29 Thread Matthias Klose
Am 29.11.2012 09:43, schrieb Richard Biener: Do we need to consider GNU ld and gold coming from different binutils versions and thus do we need two sets of linker feature tests at configure time? both GNU ld and gold are built from the same sources. So it is likely that they come from the same

Re: [PATCH] Don't bypass blocks with multiple latch edges (PR middle-end/54838)

2012-11-29 Thread Steven Bosscher
On Thu, Nov 29, 2012 at 4:38 PM, Marek Polacek wrote: On Thu, Nov 29, 2012 at 09:34:31AM +0100, Richard Biener wrote: Definitely not - that means to not preserve loops until after cprop. The goal is to preserve loops everywhere! Yikes, sorry, it wasn't clear to me what PROP_loops really

Re: [PATCH] Support -fuse-ld=bfd and -fuse-ld=gold

2012-11-29 Thread Matthias Klose
Am 29.11.2012 10:10, schrieb Markus Trippelsdorf: Additionally, what is the rationale for this patch? IOW why isn't the following enough? x4 ~ # ln -f /usr/x86_64-pc-linux-gnu/binutils-bin/git/ld.gold /usr/x86_64-pc-linux-gnu/binutils-bin/git/ld x4 ~ # ld -v GNU gold (GNU Binutils

Re: [PATCH] Add --with-build-config=bootstrap-asan support

2012-11-29 Thread Jakub Jelinek
On Thu, Nov 22, 2012 at 12:37:47PM -0800, H.J. Lu wrote: 2012-11-21 H.J. Lu hongjiu...@intel.com * Makefile.def (target_modules): Add bootstrap=true and raw_cxx=true to libsanitizer. * configure.ac (bootstrap_target_libs): Add libsanitizer. * Makefile.in:

Re: [PATCH] Stream profile summary histogram through LTO files (issue6782131)

2012-11-29 Thread Jan Hubicka
This patch ensures that the histograms from the profile summary are streamed through the LTO files so that the working set can be computed for use in downstream optimizations. Bootstrapped and tested on x86_64-unknown-linux-gnu. Ok for trunk? Thanks, Teresa 2012-11-28 Teresa Johnson

Re: [PATCH] Stream profile summary histogram through LTO files (issue6782131)

2012-11-29 Thread Teresa Johnson
On Thu, Nov 29, 2012 at 8:17 AM, Jan Hubicka hubi...@ucw.cz wrote: This patch ensures that the histograms from the profile summary are streamed through the LTO files so that the working set can be computed for use in downstream optimizations. Bootstrapped and tested on

Re: [PATCH] Stream profile summary histogram through LTO files (issue6782131)

2012-11-29 Thread Jan Hubicka
I assume you mean use the bitpack streaming functionality used other places in lto-cgraph.c, and not change the way it is done in gcov-io.c where this was cloned from? I will change the lto-cgraph.c code to use the bitpacking. Yes, we don't have the facility on gcov-io, so we need to do

Re: [PATCH] Don't bypass blocks with multiple latch edges (PR middle-end/54838)

2012-11-29 Thread Marek Polacek
On Thu, Nov 29, 2012 at 04:50:19PM +0100, Steven Bosscher wrote: 2012-11-29 Marek Polacek PR middle-end/54838 * cprop.c (bypass_block): Set header and latch to NULL when BB has more than one latch edge. (n_latches): New variable. You don't have to

Re: [PATCH] Fix for PR55492 : __atomic_load doesn't match ACQUIRE memory model

2012-11-29 Thread Andrew MacLeod
On 11/29/2012 10:42 AM, Yvan Roux wrote: Hi, on ARMv7, the code generated for the __atomic_load builtins in the __ATOMIC_ACQUIRE memory model, puts a memory barrier before the load, whereas the semantic of the acquire memory model implies a barrier after. The issue seems to be in

Re: [patch] Fix PR middle-end/55321

2012-11-29 Thread Richard Henderson
On 2012-11-29 06:29, Eric Botcazou wrote: PR middle-end/55321 * calls.c (emit_library_call_value_1): Mark as no-nonlocal if no-throw. Ok. r~

[ARM] Turning off 64bits ops in Neon and gfortran/modulo-scheduling problem

2012-11-29 Thread Christophe Lyon
Hi, I have been working on a patch to avoid using Neon for 64 bits bitops when it's too expensive to move data between core and Neon registers. Benchmarking and validation look OK on the 4.7 branch (compiler configured for thumb and hard FP) - validation on cortex-a9 board OK - bencharking shows

Re: [PATCH] Add --with-build-config=bootstrap-asan support

2012-11-29 Thread Tobias Burnus
H.J. Lu wrote: This patch adds --with-build-config=bootstrap-asan support. Tested on Linux/x86-64. OK to install? I think that patch has broken bootstrap for me. If I do a normal bootstrap, Stage1 fails with: libtool: compile: unrecognized option `-D_GNU_SOURCE' libtool: compile: Try

[PATCH Filter out -fsanitize=address if not in combined tree in lto-plugin

2012-11-29 Thread H.J. Lu
Hi, When GCC is configured with --with-build-config=bootstrap-asan, all -flto tests will fail since -fsanitize=address is used to compile liblto_plugin.so and linker isn't compiled with -fsanitize=address. This patch filters out -fsanitize=address from CFLAGS if we aren't in a combined tree with

[PATCH] Filter out -fsanitize=address if not in combined tree in libiberty

2012-11-29 Thread H.J. Lu
Hi, When GCC is configured with --with-build-config=bootstrap-asan, all -flto tests will fail since -fsanitize=address is used to compile host libiberty, which is used to create liblto_plugin.so, and linker isn't compiled with -fsanitize=address. This patch filters out -fsanitize=address from

[PATCH] Put a breakpoint on __asan_report_error for ASAN

2012-11-29 Thread H.J. Lu
Hi, This patch puts a breakpoint on __asan_report_error if CFLAGS contains -fsanitize=address, similar to fancy_abort and internal_error. OK to install? H.J. -- 2012-11-24 H.J. Lu hongjiu...@intel.com * configure.ac: Append gdbasan.in to .gdbinit if CFLAGS contains

Re: [PATCH] Add --with-build-config=bootstrap-asan support

2012-11-29 Thread H.J. Lu
On Thu, Nov 29, 2012 at 9:36 AM, Tobias Burnus bur...@net-b.de wrote: H.J. Lu wrote: This patch adds --with-build-config=bootstrap-asan support. Tested on Linux/x86-64. OK to install? I think that patch has broken bootstrap for me. If I do a normal bootstrap, Stage1 fails with:

Re: [PATCH] Don't bypass blocks with multiple latch edges (PR middle-end/54838)

2012-11-29 Thread Eric Botcazou
Yikes, sorry, it wasn't clear to me what PROP_loops really does. Anyway, I think I have a better fix now. The problem is just that when removing BB 4 (which was a header), we have to zap -header and -latch. We already have code for this: if (current_loops != NULL

[PATCH] Filter out -fsanitize=address from PLUGINCFLAGS

2012-11-29 Thread H.J. Lu
Hi, All plugin tests failed when GCC is configured with --with-build-config=bootstrap-asan, since linker can't find libasan at link-time. -fsanitize=address isn't needed to test plugin support. This patch filters out -fsanitize=address from PLUGINCFLAGS. OK to install? Thanks. H.J. ---

[patch, ARM] Fix pr55073

2012-11-29 Thread Richard Earnshaw
PR 55073 is a case where scheduling appears to mess up the order of instructions to the extent that they no-longer give the correct results. However, looking at the patterns, I think they are ill-defined. IIRC tied operands should tie a source to a destination, rather than a destination to a

Re: [PATCH] Add --with-build-config=bootstrap-asan support

2012-11-29 Thread Tobias Burnus
H.J. Lu wrote: Please try this: http://gcc.gnu.org/ml/gcc-patches/2012-11/msg01530.html I just did locally git log -2 -p|patch -p1 -R - and wait until you sort that out with the patch reviewers, which seemingly didn't like that patch. BTW: I am not sure whether it is possible with the

Go patch committed: Track fields with tag go:track

2012-11-29 Thread Ian Lance Taylor
This patch implements support for somewhat experimental field tracking. If a struct field has a tag go:track, the compiler will track references to the field. A new function runtime.Fieldtrack will add the names of all tracked fields to a map. The idea is that linker garbage collection will

Re: [patch prefix.c]: 4 of 7 Fix of PR target/53912 bootstrap fails using default c++ mode in stage 2 and 3 for native x86_64-w64-mingw32

2012-11-29 Thread Ian Lance Taylor
On Thu, Nov 29, 2012 at 4:09 AM, Kai Tietz ktiet...@googlemail.com wrote: ChangeLog 2012-11-29 Kai Tietz PR target/53912 * prefix.c (lookup_key): Explicit cast return-type of xmalloc/xrealloc to char *. Tested for i686-w64-mingw32, x86_64-w64-mingw32, and

Re: [PATCH] Add --with-build-config=bootstrap-asan support

2012-11-29 Thread Paolo Carlini
On 11/29/2012 06:36 PM, Tobias Burnus wrote: H.J. Lu wrote: This patch adds --with-build-config=bootstrap-asan support. Tested on Linux/x86-64. OK to install? I think that patch has broken bootstrap for me. If I do a normal bootstrap, Stage1 fails with: libtool: compile: unrecognized

Re: [patch prefix.c]: 4 of 7 Fix of PR target/53912 bootstrap fails using default c++ mode in stage 2 and 3 for native x86_64-w64-mingw32

2012-11-29 Thread Kai Tietz
Fine. Tested patch using XNEWVEC/XRESIZEVEC for this. Ok for apply? Kai Index: prefix.c === --- prefix.c(Revision 193939) +++ prefix.c(Arbeitskopie) @@ -157,12 +157,12 @@ lookup_key (char *key) } size = 32; - dst

[PATCH] Use explicit -I for libstdc++-v3 header files

2012-11-29 Thread H.J. Lu
Hi, Since libsanitizer is used for bootstrap and compiled with raw_cxx, we need to use explicit -I for libstdc++-v3 header files in libsanitizer. Otherwise, we will get libtool: compile: unrecognized option `-D_GNU_SOURCE' libtool: compile: Try `libtool --help' for more information. This patch

Re: [PATCH, generic] Fix for define_subst

2012-11-29 Thread Richard Henderson
On 2012-11-29 00:58, Michael Zolotukhin wrote: 2012-11-29 Michael Zolotukhin michael.v.zolotuk...@intel.com * gensupport.c (maybe_eval_c_test): Remove not-null check for expr. * read-rtl.c (apply_iterators): Initialize condition with instead of NULL. Ok. r~

Re: [patch prefix.c]: 4 of 7 Fix of PR target/53912 bootstrap fails using default c++ mode in stage 2 and 3 for native x86_64-w64-mingw32

2012-11-29 Thread Ian Lance Taylor
On Thu, Nov 29, 2012 at 10:27 AM, Kai Tietz ktiet...@googlemail.com wrote: Fine. Tested patch using XNEWVEC/XRESIZEVEC for this. Ok for apply? This is OK with a ChangeLog entry. Thanks. Ian Index: prefix.c === --- prefix.c

Re: [PATCH] Add --with-build-config=bootstrap-asan support

2012-11-29 Thread H.J. Lu
On Thu, Nov 29, 2012 at 10:24 AM, Paolo Carlini paolo.carl...@oracle.com wrote: On 11/29/2012 06:36 PM, Tobias Burnus wrote: H.J. Lu wrote: This patch adds --with-build-config=bootstrap-asan support. Tested on Linux/x86-64. OK to install? I think that patch has broken bootstrap for me.

Re: [PATCH] Vtable pointer verification, gcc changes (patch 2 of 2)

2012-11-29 Thread Jason Merrill
On 11/16/2012 01:21 PM, Caroline Tice wrote: -start_objects (int method_type, int initp) +start_objects (int method_type, int initp, const char *extra_name) I don't think we want to mess with start_objects and such here. Can't you just use DECL_STATIC_CONSTRUCTOR (decl) = 1;

Re: [PATCH] Use explicit -I for libstdc++-v3 header files

2012-11-29 Thread Andrew Pinski
On Thu, Nov 29, 2012 at 10:30 AM, H.J. Lu hongjiu...@intel.com wrote: Hi, Since libsanitizer is used for bootstrap and compiled with raw_cxx, we need to use explicit -I for libstdc++-v3 header files in libsanitizer. Otherwise, we will get libtool: compile: unrecognized option

Re: [PATCH, RFC] Dumping expanded MD files

2012-11-29 Thread Richard Henderson
On 2012-11-28 23:22, Michael Zolotukhin wrote: +.PHONY: mddump +mddump: $(BUILD_RTL) $(MD_DEPS) build/genmddump$(build_exeext) + $(RUN_GEN) build/genmddump$(build_exeext) $(md_file) 2 tmp-mddump.md Why in the world are you dumping to stderr instead of stdout? r~

Re: [patch] PR middle-end/55401: uninstrument relevant path in TM clone

2012-11-29 Thread Richard Henderson
On 2012-11-28 17:05, Aldy Hernandez wrote: Author: Aldy Hernandez al...@redhat.com Date: Wed Nov 28 18:17:33 2012 -0600 PR middle-end/55401 * trans-mem.c (get_tm_region_blocks): Exclude uninstrumented blocks from vector if requested. (collect_bb2reg): Pass new

Re: [PATCH] Use explicit -I for libstdc++-v3 header files

2012-11-29 Thread H.J. Lu
On Thu, Nov 29, 2012 at 10:34 AM, Andrew Pinski pins...@gmail.com wrote: On Thu, Nov 29, 2012 at 10:30 AM, H.J. Lu hongjiu...@intel.com wrote: Hi, Since libsanitizer is used for bootstrap and compiled with raw_cxx, we need to use explicit -I for libstdc++-v3 header files in libsanitizer.

Re: [PATCH] Use explicit -I for libstdc++-v3 header files

2012-11-29 Thread Andrew Pinski
On Thu, Nov 29, 2012 at 10:40 AM, H.J. Lu hjl.to...@gmail.com wrote: On Thu, Nov 29, 2012 at 10:34 AM, Andrew Pinski pins...@gmail.com wrote: On Thu, Nov 29, 2012 at 10:30 AM, H.J. Lu hongjiu...@intel.com wrote: Hi, Since libsanitizer is used for bootstrap and compiled with raw_cxx, we need

patch to fix PR55456

2012-11-29 Thread Vladimir Makarov
The following patch fixes gcc.gnu.org/bugzilla/show_bug.cgi?id=55456 The patch was successfully bootstrapped and tested on x86/x86-64. Committed as rev. 193948. 2012-11-28 Vladimir Makarov vmaka...@redhat.com PR middle-end/55456 * lra-int.h (lra_new_regno_start): New

Re: [PATCH] Add --with-build-config=bootstrap-asan support

2012-11-29 Thread H.J. Lu
On Thu, Nov 29, 2012 at 10:32 AM, H.J. Lu hjl.to...@gmail.com wrote: On Thu, Nov 29, 2012 at 10:24 AM, Paolo Carlini paolo.carl...@oracle.com wrote: On 11/29/2012 06:36 PM, Tobias Burnus wrote: H.J. Lu wrote: This patch adds --with-build-config=bootstrap-asan support. Tested on

Re: [PATCH, RFC] Dumping expanded MD files

2012-11-29 Thread Richard Henderson
On 2012-11-28 23:22, Michael Zolotukhin wrote: + debug_rtx (desc); ... oh, I see. You wanted dump_rtx here, so you can output to stdout. r~

Re: [PATCH] Add --with-build-config=bootstrap-asan support

2012-11-29 Thread Jakub Jelinek
On Thu, Nov 29, 2012 at 07:24:38PM +0100, Paolo Carlini wrote: On 11/29/2012 06:36 PM, Tobias Burnus wrote: H.J. Lu wrote: This patch adds --with-build-config=bootstrap-asan support. Tested on Linux/x86-64. OK to install? I think that patch has broken bootstrap for me. If I do a normal

Re: [PATCH] Add --with-build-config=bootstrap-asan support

2012-11-29 Thread Andrew Pinski
On Thu, Nov 29, 2012 at 11:03 AM, H.J. Lu hjl.to...@gmail.com wrote: On Thu, Nov 29, 2012 at 10:32 AM, H.J. Lu hjl.to...@gmail.com wrote: On Thu, Nov 29, 2012 at 10:24 AM, Paolo Carlini paolo.carl...@oracle.com wrote: On 11/29/2012 06:36 PM, Tobias Burnus wrote: H.J. Lu wrote: This patch

Re: [PATCH] Add --with-build-config=bootstrap-asan support

2012-11-29 Thread H.J. Lu
On Thu, Nov 29, 2012 at 11:06 AM, Jakub Jelinek ja...@redhat.com wrote: On Thu, Nov 29, 2012 at 07:24:38PM +0100, Paolo Carlini wrote: On 11/29/2012 06:36 PM, Tobias Burnus wrote: H.J. Lu wrote: This patch adds --with-build-config=bootstrap-asan support. Tested on Linux/x86-64. OK to

C++ PATCH for c++/53137 ('this' capture in templates)

2012-11-29 Thread Jason Merrill
My earlier semi-fix for this issue failed to update LAMBDA_EXPR_THIS_CAPTURE to point to the proxy variable, so uses ended up with an unattached FIELD_DECL. Fixed thus. Tested x86_64-pc-linux-gnu, applying to trunk and 4.7. commit 73e140fd81aa10994184bb8edfe9cfafdc1499a9 Author: Jason Merrill

Re: [patch] RFA: more fixes for PR55006

2012-11-29 Thread Dominique Dhumieres
Of course, I also verified that the darwin10 and sparc issues that spawned this long thread are fixed. Confirmed: bootstrapped on top of r193939, aermod.f90 is now correctly compiled. Thanks, Dominique

Re: [PATCH] Add --with-build-config=bootstrap-asan support

2012-11-29 Thread H.J. Lu
On Thu, Nov 29, 2012 at 11:06 AM, Jakub Jelinek ja...@redhat.com wrote: On Thu, Nov 29, 2012 at 07:24:38PM +0100, Paolo Carlini wrote: On 11/29/2012 06:36 PM, Tobias Burnus wrote: H.J. Lu wrote: This patch adds --with-build-config=bootstrap-asan support. Tested on Linux/x86-64. OK to

Re: [RFA 4/8] validate_failures.py: rename --manifest to --manifest_path

2012-11-29 Thread Doug Evans
Diego Novillo writes: On Sat, Nov 24, 2012 at 5:50 PM, Doug Evans d...@google.com wrote: Hi. This fourth patch renames option --manifest to --manifest_path. I have a later patch that adds a --manifest_name option, making --manifest too confusing/ambiguous. Ok to check in?

  1   2   >