[patch libffi]: Make sure code is position-independent for x64 targets

2013-03-22 Thread Kai Tietz
Hi, this patch fixes the Win64-code so that we use only pc-relative addressing. This is of importance if code gets linked to an pe-image with an image-base above 2GB. ChangeLog 2013-03-22 Kai Tietz kti...@redhat.com * src/x86/win64.S: Make use of ffi_closure_win64_inner

[patch i386]: Treat dllimported symbol as external in predicates

2013-03-22 Thread Kai Tietz
Hi, A dllimported symbol is always external. So treat that proper in local_symbolic_operand. ChangeLog 2013-03-22 Kai Tietz kti...@redhat.com * config/i386/predicates.md (local_symbolic_operand): Interprete dll-imported symbols as none-local. Tested for x86_64-w64-mingw32,

[patch mingw]: Make POSIX-printf for mingw-based builds default

2013-03-22 Thread Kai Tietz
Hi, this patch enables the POSIX-printf variant for mingw-hosts by default. ChangeLog 2013-03-22 Kai Tietz kti...@redhat.com * config/i386/xm-mingw32.h (__USE_MINGW_ANSI_STDIO): Enable POSIX-printf for mingw-hosted builds. Tested for i686-w64-mingw32, and

[patch java]: Avoid looping over the end_params_node in put_decl_node

2013-03-22 Thread Kai Tietz
Hi, this patch makes sure we don't walk in put_decl_node of the end_params_node element. ChangeLog 2013-03-22 Kai Tietz kti...@redhat.com * lang.c (put_decl_node): Don't iterate over end_params_node. Tested for i686-w64-mingw32, x86_64-w64-mingw32, and x86_64-unknown-linux-gnu. Ok

Re: [Patch, libfortran] PR 52512 - Cannot match namelist object name

2013-03-22 Thread Janne Blomqvist
On Fri, Mar 22, 2013 at 12:52 AM, Tilo Schwarz t...@tilo-schwarz.de wrote: Hi, this patch fixes PR 52512. Built and regtested on Linux 3.2.0-4-686-pae. Ok for trunk (4.9). Thanks for the patch! PS: Common procedure is to report the target triplet you regtested on rather than the Linux

[patch testsuite]: g++.dg

2013-03-22 Thread Kai Tietz
Hi, this patch fixes an LLP64 issue in g++.dg's testsuite. ChangeLog 2013-03-22 Kai Tietz kti...@redhat.com * g++.dg/torture/20121105-1.C: Adjust for LLP64 targets. Ok for apply? Regards, Kai Index: gcc/testsuite/g++.dg/torture/20121105-1.C

Re: [Patch, fortran, 4.9] Use bool type instead gfc_try

2013-03-22 Thread N.M. Maclaren
On Mar 21 2013, Joseph S. Myers wrote: now that the Fortran frontend is C++ we can use the primitive bool type instead of inventing our own. Well, C99's bool (_Bool) was already used before. ... Er, that is making a serious mistake or, at least, running the risk of one. C++'s bool

Re: [Patch, fortran, 4.9] Use bool type instead gfc_try

2013-03-22 Thread Miles Bader
N.M. Maclaren n...@cam.ac.uk writes: That is another matter entirely. The code of gcc/gfortran is supposed to be compilable with other compilers, and it is foolish to make unnecessary assumptions by relying on undefined behaviour. The simple facts are that C++ does NOT define bool to be

[patch libstdc++ testsuite]: Add new option dg-additional-options and make use of it for mingw targets

2013-03-22 Thread Kai Tietz
Hi, this patch adds option dg-additional-options for libstdc++-v3's testsuite and make use of this option for some mingw-tests. Those tests are just possible for pe-coff for static-library use. The shared version for pe-coff is finally linked and therefore no override of operators is possible

[patch libgcc]: Prepare cygwin.S file to share code with cygwin64

2013-03-22 Thread Kai Tietz
Hi, this patch replaces use of _WIN64 by __x86_64__ so mingw x64 and cygwin x64 version can share same source. ChangeLog 2013-03-22 Kai Tietz kti...@redhat.com * config/i386/cygwin.S: Replace use of _WIN64 by __x86_64__. Tested for x86_64-w64-mingw32, x86_64-pc-cygwin, and

[patch libgcc]: Adjust cygming-crtbegin code to use weak

2013-03-22 Thread Kai Tietz
Hi, this change is actual used by cygwin and is required for upcoming x64 cygwin target. ChangeLog 2013-03-22 Kai Tietz kti...@redhat.com * config/i386/cygming-crtbegin.c (__register_frame_info): Make weak. (__deregister_frame_info): Likewise. Tested for i686-pc-cygwin, and

[patch cygwin64]: Add and adjust some initial sources for x64 cygwin

2013-03-22 Thread Kai Tietz
Hi, Hi, the first part of required code-changes for upcoming cygwin x64 target. ChangeLog 2013-03-22 Kai Tietz kti...@redhat.com * config/i386/cygwin-stdint.h: Add support for cygwin x64 target. * config/i386/t-cygwin-w64: New file. * config/i386/cygwin-w64.h: New

Re: [patch java]: Avoid looping over the end_params_node in put_decl_node

2013-03-22 Thread Andrew Haley
On 03/22/2013 08:13 AM, Kai Tietz wrote: Tested for i686-w64-mingw32, x86_64-w64-mingw32, and x86_64-unknown-linux-gnu. Ok for apply? Yes, thanks. Andrew.

Re: [Patch, fortran, 4.9] Use bool type instead gfc_try

2013-03-22 Thread N.M. Maclaren
On Mar 22 2013, Miles Bader wrote: That is another matter entirely. The code of gcc/gfortran is supposed to be compilable with other compilers, and it is foolish to make unnecessary assumptions by relying on undefined behaviour. The simple facts are that C++ does NOT define bool to be

Re: [patch libffi]: Make sure code is position-independent for x64 targets

2013-03-22 Thread Andrew Haley
On 03/22/2013 07:42 AM, Kai Tietz wrote: Tested for x86_64-w64-mingw32, and for upcoming x86_64-pc-cygwin target. Ok for apply? Yes, that's fine. Andrew.

Re: C++ PATCH: use identifier_p instead of naked TREE_CODE == IDENTIFIER_NODE

2013-03-22 Thread Richard Biener
On Fri, Mar 22, 2013 at 4:50 AM, Gabriel Dos Reis g...@axiomatics.org wrote: This patch introduces identified_p (t) in lieu of TREE_CODE (t) == IDENTIFIER_NODE Generally we have macros like IDENTIFIER_P for this kind of checks. in the C++ front-end. identifier_p is effectively

[patch i386 windows]: Fix PR/52790 also required for workig upcoming cygwin x64 target

2013-03-22 Thread Kai Tietz
Hi, this patch fixes PR/52790 and supports for x64 Windows targets the use of large and medium code-model. This feature is required for upcoming new cygwin x64 target, which uses full 48-bit available address-space of x64 Windows. The cygwin-target depends on pseudo-relocation-feature, which

Re: C++ PATCH: use identifier_p instead of naked TREE_CODE == IDENTIFIER_NODE

2013-03-22 Thread Jakub Jelinek
On Fri, Mar 22, 2013 at 10:21:05AM +0100, Richard Biener wrote: On Fri, Mar 22, 2013 at 4:50 AM, Gabriel Dos Reis g...@axiomatics.org wrote: This patch introduces identified_p (t) in lieu of TREE_CODE (t) == IDENTIFIER_NODE Generally we have macros like IDENTIFIER_P for this kind of

Re: [patch] Fix ICE during RTL expansion at -O1

2013-03-22 Thread Richard Biener
On Thu, Mar 21, 2013 at 5:24 PM, Eric Botcazou ebotca...@adacore.com wrote: Hi, this fixes an ICE on the mainline at -O1: eric@polaris:~/gnat/bugs/M129-026 ~/install/gcc/bin/gcc -S p.adb -O +===GNAT BUG DETECTED==+ | 4.9.0 20130320

Re: [patch] df-scan: split df_insn_delete for clearer dumps and better speed

2013-03-22 Thread Steven Bosscher
On Mon, Mar 18, 2013 at 1:01 AM, Steven Bosscher wrote: On Thu, Feb 21, 2013 at 1:33 PM, Paolo Bonzini wrote: someone else DF aware anyway. If it is 4.9 material perhaps it is simpler to place it after the patches that kill the basic block argument. Here is the updated patch.

Re: [patch] df-scan: split df_insn_delete for clearer dumps and better speed

2013-03-22 Thread Paolo Bonzini
Il 18/03/2013 01:01, Steven Bosscher ha scritto: someone else DF aware anyway. If it is 4.9 material perhaps it is simpler to place it after the patches that kill the basic block argument. Here is the updated patch. Bootstrappedtested on powerpc64-unknown-linux-gnu. OK for trunk? Ok.

Re: [PATCH] Fix PR56434

2013-03-22 Thread Jakub Jelinek
On Fri, Mar 22, 2013 at 11:06:53AM +0100, Richard Biener wrote: This fixes PR56434 - the use of BIGGEST_ALIGNMENT to annotate the pointer returned by malloc is wrong - BIGGEST_ALIGNMENT has nothing to do with the alignment guaranteed by the ABI for allocated memory. For example on x86_64 it

[PATCH] Update changes.html about -Wno-aggressive-loop-optimizations

2013-03-22 Thread Jakub Jelinek
Hi! This new warning switch hasn't been mentioned in changes.html, the following patch fixes that. Is this ok, or do you have better wording? --- htdocs/gcc-4.8/changes.html 14 Mar 2013 01:13:56 - 1.108 +++ htdocs/gcc-4.8/changes.html 22 Mar 2013 10:22:37 - @@ -32,7 +32,12 @@ the

Re: [PATCH] Fix PR56434

2013-03-22 Thread Richard Biener
On Fri, 22 Mar 2013, Jakub Jelinek wrote: On Fri, Mar 22, 2013 at 11:06:53AM +0100, Richard Biener wrote: This fixes PR56434 - the use of BIGGEST_ALIGNMENT to annotate the pointer returned by malloc is wrong - BIGGEST_ALIGNMENT has nothing to do with the alignment guaranteed by the ABI

Re: [Patch, fortran, 4.9] Use bool type instead gfc_try

2013-03-22 Thread Tobias Burnus
N.M. Maclaren wrote: On Mar 22 2013, Miles Bader wrote: That is another matter entirely. The code of gcc/gfortran is supposed to be compilable with other compilers, and it is foolish to make unnecessary assumptions by relying on undefined behaviour. The simple facts are that C++ does NOT

New German PO file for 'gcc' (version 4.8-b20130224)

2013-03-22 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the German team of translators. The file is available at: http://translationproject.org/latest/gcc/de.po (This file, 'gcc-4.8-b20130224.de.po',

Re: [Patch, fortran, 4.9] Use bool type instead gfc_try

2013-03-22 Thread N.M. Maclaren
On Mar 22 2013, Tobias Burnus wrote: The front end and the backend are both compiled with the same compiler and in the same binary. Even without bootstrapping, trying to compile them with different compilers, will require some heavy editing of makefiles. Thus, it seems to be extremely

[PATCH] Re-organize toplevel LIM dependence query

2013-03-22 Thread Richard Biener
This re-organizes the toplevel LIM dependence query which is to ask whether a reference is independent in a whole loop nest. The patch makes it compute the answer for all sub-loops of the nest and compute an overall answer based on the answers from the sub-loops. This avoids repeated dependence

RE: [PATCH][ARM][2/2] Load-acquire, store-release atomics in AArch32 ARMv8

2013-03-22 Thread Kyrylo Tkachov
Ping? http://gcc.gnu.org/ml/gcc-patches/2013-02/msg00023.html I thought this was ok'd for 4.9 but I can't seem to find the ok email in the archives Thanks, Kyrill -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Kyrylo

RE: [PATCH][ARM][1/2] Load-acquire, store-release atomics in AArch32 ARMv8

2013-03-22 Thread Kyrylo Tkachov
Ping? http://gcc.gnu.org/ml/gcc-patches/2013-01/msg01441.html I thought this was ok'd for 4.9 but I can't seem to find the ok email in the archives. Thanks, Kyrill -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Kyrylo

[PATCH] Remove LIM all_refs_in_loop bitmaps

2013-03-22 Thread Richard Biener
This removes the all_refs_in_loop bitmaps. They add a O(loop-depth) memory cost to the refs_in_loop bitmaps. It removes the last user, a query that is used to avoid walking subloops. But that query is costly compared to just walking the subloops. No observable memory usage difference for

[PATCH] Remove the reference dependence check cache from LIM

2013-03-22 Thread Richard Biener
The following patch finally removes the reference dependence check cache. Its benefit is too low for its cost. LIM Compile-time for PR39326 improves from 100s to 50s and LIMs memory use drops to unnoticable (from 915MB memory use after LIM to 380MB memory use after LIM). Bootstrapped on

Re: C++ PATCH: use identifier_p instead of naked TREE_CODE == IDENTIFIER_NODE

2013-03-22 Thread Gabriel Dos Reis
Jakub Jelinek ja...@redhat.com writes: | On Fri, Mar 22, 2013 at 10:21:05AM +0100, Richard Biener wrote: | On Fri, Mar 22, 2013 at 4:50 AM, Gabriel Dos Reis g...@axiomatics.org wrote: | | This patch introduces identified_p (t) in lieu of | | TREE_CODE (t) == IDENTIFIER_NODE | |

Re: C++ PATCH: use identifier_p instead of naked TREE_CODE == IDENTIFIER_NODE

2013-03-22 Thread Gabriel Dos Reis
Richard Biener richard.guent...@gmail.com writes: [...] | in the C++ front-end. identifier_p is effectively LANG_IDENTIFIER_CAST | except that it returns a typed pointer instead of a boolean value. | | Hm? So you are replacing TREE_CODE (t) == IDENTIFIER_NODE | with kind-of

Re: [PATCH] Fix for movmode_internal pattern

2013-03-22 Thread Michael Zolotukhin
Do you have a testcase to show there is a problem? The misaligned case should only be generated from ix86_avx256_split_vector_move_misalign. I faced it when playing with optimized expanding of memmov (with vector instructions). There I generated v2di-move with emit_strmov, which later used

Re: C++ PATCH: use identifier_p instead of naked TREE_CODE == IDENTIFIER_NODE

2013-03-22 Thread Richard Biener
On Fri, Mar 22, 2013 at 1:44 PM, Gabriel Dos Reis g...@axiomatics.org wrote: Richard Biener richard.guent...@gmail.com writes: [...] | in the C++ front-end. identifier_p is effectively LANG_IDENTIFIER_CAST | except that it returns a typed pointer instead of a boolean value. | | Hm? So

[PATCH] Use gcc_checking_assert in as_a

2013-03-22 Thread Richard Biener
Committed as obvious. Richard. 2013-03-22 Richard Biener rguent...@suse.de * is-a.h (as_a): Use gcc_checking_assert. Index: gcc/is-a.h === --- gcc/is-a.h (revision 196955) +++ gcc/is-a.h (working copy) @@ -181,7

Re: [PATCH] Fix PR56434

2013-03-22 Thread Ian Lance Taylor
On Fri, Mar 22, 2013 at 3:27 AM, Richard Biener rguent...@suse.de wrote: I think the wrong-code fix is orthogonal to code improvements which will also trigger on the GIMPLE level (and where they will have a bigger impact). I agree. I think the patch to calls is fine unless Jakub objects.

Re: [PATCH] Fix PR56434

2013-03-22 Thread Richard Biener
On Fri, 22 Mar 2013, Ian Lance Taylor wrote: On Fri, Mar 22, 2013 at 3:27 AM, Richard Biener rguent...@suse.de wrote: I think the wrong-code fix is orthogonal to code improvements which will also trigger on the GIMPLE level (and where they will have a bigger impact). I agree. I think

[committed] Use automake no-dist for libsanitizer (PR other/43620)

2013-03-22 Thread Jakub Jelinek
Hi! Ugh, this hit us again, going to note it into releasing.html later. libsanitizer Makefiles/configure are owned by GCC, so we can use no-dist here on the trunk too. Committed as obvious. 2013-03-22 Jakub Jelinek ja...@redhat.com PR other/43620 libsanitizer/ * configure.ac

Re: C++ PATCH: use identifier_p instead of naked TREE_CODE == IDENTIFIER_NODE

2013-03-22 Thread Gabriel Dos Reis
Richard Biener richard.guent...@gmail.com writes: | On Fri, Mar 22, 2013 at 1:44 PM, Gabriel Dos Reis g...@axiomatics.org wrote: | Richard Biener richard.guent...@gmail.com writes: | | [...] | | | in the C++ front-end. identifier_p is effectively LANG_IDENTIFIER_CAST | | except that it

Re: [PATCH][ARM][2/2] Load-acquire, store-release atomics in AArch32 ARMv8

2013-03-22 Thread Richard Earnshaw
On 01/02/13 17:38, Kyrylo Tkachov wrote: Ummm... forgot the patch, sorry! -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Kyrylo Tkachov Sent: 01 February 2013 17:37 To: gcc-patches@gcc.gnu.org Cc: Ramana Radhakrishnan;

Re: [patch i386]: Treat dllimported symbol as external in predicates

2013-03-22 Thread Richard Henderson
On 03/22/2013 01:03 AM, Kai Tietz wrote: * config/i386/predicates.md (local_symbolic_operand): Interprete dll-imported symbols as none-local. Ok. r~

Re: [PATCH][ARM][1/2] Load-acquire, store-release atomics in AArch32 ARMv8

2013-03-22 Thread Richard Earnshaw
On 30/01/13 15:54, Kyrylo Tkachov wrote: Hi all, This patch implements the atomic built-ins using the new ARMv8 load-acquire and store-release instructions. They allow us to generate barrier-free code for a variety of atomic operations such as: atomic load, atomic store, atomic compare and

Re: [PATCH] Fix for movmode_internal pattern

2013-03-22 Thread H.J. Lu
On Fri, Mar 22, 2013 at 5:49 AM, Michael Zolotukhin michael.v.zolotuk...@gmail.com wrote: Do you have a testcase to show there is a problem? The misaligned case should only be generated from ix86_avx256_split_vector_move_misalign. I faced it when playing with optimized expanding of memmov

Re: [PATCH] Fix for movmode_internal pattern

2013-03-22 Thread Michael Zolotukhin
You can't use aligned vector move if alignment is unknown or not properly aligned. Yes, sure. But I just emit V2DI move for an operands, which are aligned to 64-bit (not 128-bit!) - and it's encoded into movdqa which is obviously incorrect. The compiler knows, that operands are misaligned, but

[cilkplus-merge] fix various wrapping issues and correct some docs wording

2013-03-22 Thread Aldy Hernandez
I am committing the following changes as obvious. They include some wrapping issues, as well as some minor wording issues I found while peeking around. Joseph, I am posting this as an incremental patch, to avoid posting a 7k line patch for everything. I have also asked Balaji to post

RE: FW: [PATCH] [MIPS] microMIPS gcc support

2013-03-22 Thread Moore, Catherine
-Original Message- From: Richard Sandiford [mailto:rdsandif...@googlemail.com] Sent: Thursday, March 21, 2013 8:04 PM To: Moore, Catherine Cc: gcc-patches@gcc.gnu.org; Rozycki, Maciej Subject: Re: FW: [PATCH] [MIPS] microMIPS gcc support Thanks, this is almost there now. It was

[PATCH 4/n, i386]: Merge and rewrite MMX move patterns using x64 and nox64 isa attribute

2013-03-22 Thread Uros Bizjak
Hello! Attached patch merges _rex64 MMX move patterns with base MMX move patters using x64 and nox64 isa attribute. Additionally, the patch rewrites MMX move patterns to look like DImove move pattern (keeping all decorations of various alternatives), introducing all recent improvements. The patch

Add Go notes for GCC 4.8 release

2013-03-22 Thread Ian Lance Taylor
I committed some notes on Go to the GCC 4.8 changes file. Ian Index: gcc-4.8/changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.8/changes.html,v retrieving revision 1.109 diff -u -r1.109 changes.html --- gcc-4.8/changes.html

Re: [PATCH] Fix for movmode_internal pattern

2013-03-22 Thread H.J. Lu
On Fri, Mar 22, 2013 at 9:58 AM, Michael Zolotukhin michael.v.zolotuk...@gmail.com wrote: You can't use aligned vector move if alignment is unknown or not properly aligned. Yes, sure. But I just emit V2DI move for an operands, which are aligned to 64-bit (not 128-bit!) - and it's encoded into

Re: [patch testsuite]: g++.dg

2013-03-22 Thread Mike Stump
On Mar 22, 2013, at 1:17 AM, Kai Tietz ktiet...@googlemail.com wrote: this patch fixes an LLP64 issue in g++.dg's testsuite. ChangeLog 2013-03-22 Kai Tietz kti...@redhat.com * g++.dg/torture/20121105-1.C: Adjust for LLP64 targets. Ok for apply? Ok.

Re: [patch libstdc++ testsuite]: Add new option dg-additional-options and make use of it for mingw targets

2013-03-22 Thread Mike Stump
On Mar 22, 2013, at 1:31 AM, Kai Tietz ktiet...@googlemail.com wrote: this patch adds option dg-additional-options for libstdc++-v3's testsuite and make use of this option for some mingw-tests. Those tests are just possible for pe-coff for static-library use. The shared version for pe-coff

C++ PATCH for c++/56684 (ICE with const local in template)

2013-03-22 Thread Jason Merrill
We were treating a constant VAR_DECL as non-dependent because it has a non-dependent type, even though it has a value-dependent initializer. Tested x86_64-pc-linux-gnu, applying to trunk. commit d3a8f99d0ae9caf8d344b44a8e66632181a20157 Author: Jason Merrill ja...@redhat.com Date: Fri Mar 22

[wwwdocs] 4.8.0 libstdc++ links

2013-03-22 Thread Benjamin De Kosnik
Slight update to links, reflecting more attempts by libstdc++ to blend in with the native layout and compression choices for gcc releases. Links check so seems fine. While at this, I figured I might as well do the GDL/GPL dupe for the libstdc++-api XML file, as discussed at Cauldron last year.

[PATCH] Sort targets alphabetically

2013-03-22 Thread Sebastian Huber
gcc/testsuite/ChangeLog 2013-03-22 Sebastian Huber sebastian.hu...@embedded-brains.de * gcc.c-torture/execute/builtins/builtins.exp: Sort targets alphabetically. --- gcc/testsuite/gcc.c-torture/execute/builtins/builtins.exp |4 ++-- 1 file changed, 2 insertions(+), 2

Re: [Patch, libfortran] PR 52512 - Cannot match namelist object name

2013-03-22 Thread Tilo Schwarz
On Fri, 22 Mar 2013 09:13:17 +0100, Janne Blomqvist blomqvist.ja...@gmail.com wrote: On Fri, Mar 22, 2013 at 12:52 AM, Tilo Schwarz t...@tilo-schwarz.de wrote: Hi, this patch fixes PR 52512. Built and regtested on Linux 3.2.0-4-686-pae. Ok for trunk (4.9). Thanks for the patch! PS:

Re: [PATCH v2] Fix builtin tests for RTEMS targets

2013-03-22 Thread Mike Stump
On Mar 21, 2013, at 7:08 AM, Sebastian Huber sebastian.hu...@embedded-brains.de wrote: This patch should be applied to GCC 4.8 and 4.9. Ok for trunk. Ok for 4.8.1. 4.8.0 would require RM approval I think.

Re: [patch] cilkplus array notation for C (clean, independent patchset, take 1)

2013-03-22 Thread Mike Stump
On Mar 21, 2013, at 4:33 PM, Aldy Hernandez al...@redhat.com wrote: I can look into this later on, but this problem happened even when I replaced cilkplus' compile.exp, errors.exp, and execute.exp into just an exit. So it seems unrelated to the cilk patch set. Ah, I withdraw my objection.

[GOOGLE] Fix the error when backporting r193857 from trunk

2013-03-22 Thread Dehao Chen
Hi, This patch fixes the error in r193861, which backported r193857 from trunk. Bootstrapped and passed all gcc regression tests. Is it okay for google-4_7? Thanks, Dehao

Re: [GOOGLE] Fix the error when backporting r193857 from trunk

2013-03-22 Thread Dehao Chen
forgot to attach the patch... Index: gcc/ipa-prop.c === --- gcc/ipa-prop.c (revision 196984) +++ gcc/ipa-prop.c (working copy) @@ -2473,7 +2473,6 @@ base = gimple_call_arg (stmt, adj-base_index); loc = DECL_P (base) ?

Re: [PATCH v2] Fix builtin tests for RTEMS targets

2013-03-22 Thread Mike Stump
On Mar 21, 2013, at 7:08 AM, Sebastian Huber sebastian.hu...@embedded-brains.de wrote: This patch should be applied to GCC 4.8 and 4.9. Ok for trunk. Ok for 4.8.1. 4.8.0 would require RM approval I think.

Re: [GOOGLE] Fix the error when backporting r193857 from trunk

2013-03-22 Thread Xinliang David Li
ok. David On Fri, Mar 22, 2013 at 3:58 PM, Dehao Chen de...@google.com wrote: forgot to attach the patch... Index: gcc/ipa-prop.c === --- gcc/ipa-prop.c (revision 196984) +++ gcc/ipa-prop.c (working copy) @@ -2473,7 +2473,6

[patch, mips] Patch to control the use of integer madd/msub instructions

2013-03-22 Thread Steve Ellcey
While testing GCC on a 74k MIPS chip I noticed that by default the -mtune=74k* flags cause GCC to not use the integer madd/msub instructions. According to the checkin comments these were found to cause a performance hit over using individual mult and add/sub instructions. I think there are some

Re: [patch libgcc]: Adjust cygming-crtbegin code to use weak

2013-03-22 Thread Dave Korn
On 22/03/2013 08:44, Kai Tietz wrote: Hi, this change is actual used by cygwin and is required for upcoming x64 cygwin target. ChangeLog 2013-03-22 Kai Tietz kti...@redhat.com * config/i386/cygming-crtbegin.c (__register_frame_info): Make weak.

Re: [patch cygwin64]: Add and adjust some initial sources for x64 cygwin

2013-03-22 Thread Dave Korn
On 22/03/2013 09:00, Kai Tietz wrote: (LIBGCJ_SONAME): Make name minor-build-version dependent. Tested for i686-pc-cygwin, and x86_64-pc-cygwin. Dave, please especially a look to LIBGCJ_SONAME change. I think we should include MAJOR_VERSION here, too. Ok for apply? I'm not sure

Re: [patch libgcc]: Adjust cygming-crtbegin code to use weak

2013-03-22 Thread Kai Tietz
2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: On 22/03/2013 08:44, Kai Tietz wrote: Hi, this change is actual used by cygwin and is required for upcoming x64 cygwin target. ChangeLog 2013-03-22 Kai Tietz kti...@redhat.com * config/i386/cygming-crtbegin.c

Re: [patch cygwin64]: Add and adjust some initial sources for x64 cygwin

2013-03-22 Thread Dave Korn
On 22/03/2013 09:00, Kai Tietz wrote: (CXX_WRAP_SPEC_LIST): Undefine before define. @@ -73,6 +82,7 @@ /* To implement C++ function replacement we always wrap the cxx malloc-like operators. See N2800 #17.6.4.6 [replacement.functions] */ +#undef CXX_WRAP_SPEC_LIST #define

Re: [patch cygwin64]: Add and adjust some initial sources for x64 cygwin

2013-03-22 Thread Kai Tietz
2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: On 22/03/2013 09:00, Kai Tietz wrote: (LIBGCJ_SONAME): Make name minor-build-version dependent. Tested for i686-pc-cygwin, and x86_64-pc-cygwin. Dave, please especially a look to LIBGCJ_SONAME change. I think we should include

Re: [patch libgcc]: Adjust cygming-crtbegin code to use weak

2013-03-22 Thread Dave Korn
On 23/03/2013 00:08, Kai Tietz wrote: 2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: Also, can you explain the motivation for this change? I don't see how it's going to work right; from what I remember, we don't have weak definitions in PE-COFF, just weak references. How does the correct

Re: [patch cygwin64]: Add and adjust some initial sources for x64 cygwin

2013-03-22 Thread Kai Tietz
2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: On 22/03/2013 09:00, Kai Tietz wrote: (CXX_WRAP_SPEC_LIST): Undefine before define. @@ -73,6 +82,7 @@ /* To implement C++ function replacement we always wrap the cxx malloc-like operators. See N2800 #17.6.4.6

Re: [patch cygwin64]: Add and adjust some initial sources for x64 cygwin

2013-03-22 Thread Dave Korn
On 23/03/2013 00:16, Kai Tietz wrote: welcome, too. It would be even better if we could rethink actual the need of loading java-library within libgcc's cygwin's/mingw's crtbegin at all. I am actual not that sure, if we need this at all. Somebody has to register the default classes at

Re: [patch cygwin64]: Add and adjust some initial sources for x64 cygwin

2013-03-22 Thread Kai Tietz
2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: On 23/03/2013 00:16, Kai Tietz wrote: welcome, too. It would be even better if we could rethink actual the need of loading java-library within libgcc's cygwin's/mingw's crtbegin at all. I am actual not that sure, if we need this at all.

Re: [patch]: Add configure and required host/target definitions for new cygwin x64 target

2013-03-22 Thread Dave Korn
On 22/03/2013 09:56, Kai Tietz wrote: Hi, this patch adds required configure changes for new cygwin x64 target. Index: gcc/configure.ac === --- gcc/configure.ac (Revision 196898) +++ gcc/configure.ac (Arbeitskopie) @@

Re: [patch libgcc]: Adjust cygming-crtbegin code to use weak

2013-03-22 Thread Kai Tietz
2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: On 23/03/2013 00:08, Kai Tietz wrote: 2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: Also, can you explain the motivation for this change? I don't see how it's going to work right; from what I remember, we don't have weak definitions in

Re: [patch cygwin64]: Add and adjust some initial sources for x64 cygwin

2013-03-22 Thread Dave Korn
On 23/03/2013 00:24, Kai Tietz wrote: 2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: On 23/03/2013 00:16, Kai Tietz wrote: welcome, too. It would be even better if we could rethink actual the need of loading java-library within libgcc's cygwin's/mingw's crtbegin at all. I am actual not

Re: [patch libgcc]: Adjust cygming-crtbegin code to use weak

2013-03-22 Thread Dave Korn
On 23/03/2013 00:27, Kai Tietz wrote: 2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: On 23/03/2013 00:08, Kai Tietz wrote: 2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: Also, can you explain the motivation for this change? I don't see how it's going to work right; from what I

Re: [patch]: Add configure and required host/target definitions for new cygwin x64 target

2013-03-22 Thread Kai Tietz
2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: On 22/03/2013 09:56, Kai Tietz wrote: Hi, this patch adds required configure changes for new cygwin x64 target. Index: gcc/configure.ac === --- gcc/configure.ac (Revision

Re: [patch] Use NOTE_P, CALL_P, etc. instead of GET_CODE in a bunch of files

2013-03-22 Thread Steven Bosscher
On Fri, Mar 22, 2013 at 8:09 PM, Steven Bosscher wrote: Hello, This is an almost completely mechanical replacement of GET_CODE(thing) == ... with the equivalent predicate macro from rtl.h. This particular set of files fell victim to my plans for GCC 4.9 to make JUMP_TABLE_DATA_P a separate

Re: [patch cygwin64]: Add and adjust some initial sources for x64 cygwin

2013-03-22 Thread Kai Tietz
2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: On 23/03/2013 00:24, Kai Tietz wrote: 2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: On 23/03/2013 00:16, Kai Tietz wrote: welcome, too. It would be even better if we could rethink actual the need of loading java-library within libgcc's

Re: [patch]: Add configure and required host/target definitions for new cygwin x64 target

2013-03-22 Thread Dave Korn
On 23/03/2013 00:41, Kai Tietz wrote: 2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: On 22/03/2013 09:56, Kai Tietz wrote: Hi, this patch adds required configure changes for new cygwin x64 target. Index: gcc/configure.ac ===

Re: [patch cygwin64]: Add and adjust some initial sources for x64 cygwin

2013-03-22 Thread Dave Korn
On 23/03/2013 00:57, Kai Tietz wrote: 2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: On 23/03/2013 00:24, Kai Tietz wrote: 2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: On 23/03/2013 00:16, Kai Tietz wrote: welcome, too. It would be even better if we could rethink actual the need of

RE: [patch] cilkplus array notation for C (clean, independent patchset, take 1)

2013-03-22 Thread Iyer, Balaji V
From: gcc-patches-ow...@gcc.gnu.org [gcc-patches-ow...@gcc.gnu.org] on behalf of Mike Stump [m...@mrs.kithrup.com] Sent: Friday, March 22, 2013 6:36 PM To: Aldy Hernandez Cc: Jakub Jelinek; Jeff Law; Joseph S. Myers; Iyer, Balaji V; gcc-patches Subject:

Re: [patch] Use NOTE_P, CALL_P, etc. instead of GET_CODE in a bunch of files

2013-03-22 Thread Jeff Law
On 03/22/2013 06:44 PM, Steven Bosscher wrote: On Fri, Mar 22, 2013 at 8:09 PM, Steven Bosscher wrote: Hello, This is an almost completely mechanical replacement of GET_CODE(thing) == ... with the equivalent predicate macro from rtl.h. This particular set of files fell victim to my plans for

PATCH: Restore GCC 4.7 bootstrap on x86

2013-03-22 Thread H.J. Lu
BOOL_BITFIELD isn't defined when building libgcc. I checked in this patch to restore GCC 4.7 bootstrap on x86. GCC 4.6 is OK. H.J. --- Index: ChangeLog === --- ChangeLog (revision 196998) +++ ChangeLog (working copy) @@ -1,5

Re: [patch] cilkplus array notation for C (clean, independent patchset, take 1)

2013-03-22 Thread Mike Stump
On Mar 22, 2013, at 6:36 PM, Iyer, Balaji V balaji.v.i...@intel.com wrote: I can confirm that renaming scripts to something unique fixed the issue. That's what others have said, I've not see it first hand.