[COMMITTED] Update rs6000.c's pass_analyze_swaps to use rtx_insn [was Re: [PATCH 029/236] rtl_data.x_parm_birth_insn is an insn]

2014-08-26 Thread David Malcolm
On Wed, 2014-08-13 at 11:13 -0600, Jeff Law wrote: > On 08/13/14 11:08, David Malcolm wrote: > > On Wed, 2014-08-13 at 07:44 -0600, Jeff Law wrote: > >> On 08/06/14 11:20, David Malcolm wrote: > >>> gcc/ > >>> * function.h (struct rtl_data): Strengthen field > >>> "x_parm_birth_insn" from rtx t

Re: Enable EBX for x86 in 32bits PIC code

2014-08-26 Thread Ilya Enkovich
2014-08-25 19:08 GMT+04:00 Vladimir Makarov : > On 2014-08-22 8:21 AM, Ilya Enkovich wrote: >> >> Hi, >> >> On Cauldron 2014 we had a couple of talks about relaxation of ebx usage in >> 32bit PIC mode. It was decided that the best approach would be to not fix >> ebx register, use speudo register f

Re: [PATCH][match-and-simplify] Auto-guess conversion types

2014-08-26 Thread Richard Biener
On Thu, 21 Aug 2014, Marc Glisse wrote: > On Thu, 21 Aug 2014, Richard Biener wrote: > > > 2014-08-21 Richard Biener > > > > * match.pd ((T1)(~(T2) X) -> ~(T1) X): Paste all comment > > from fold-const.c, fix simplification result. > > > > Index: gcc/match.pd > >

Re: [PATCH] Fix condition in is_aligning_offset (PR c/61271)

2014-08-26 Thread Richard Biener
On Thu, Aug 21, 2014 at 5:52 PM, Marek Polacek wrote: > This is one of the issues that -Wlogical-not-parentheses detected. > Interestingly, this code has been added in 2002 (!). I believe the > logical not there should be just removed; the comment above it says > /* We must now have a BIT_AND_E

Re: [PATCH. libstdc++] Use the correct C++14 __cplusplus value (201402L). Added C++1z to the preprocessor.

2014-08-26 Thread Paolo Carlini
Hi, On 08/24/2014 12:37 AM, Ed Smith-Rowland wrote: With revision 214400 we have the C++14 value of __cplusplus set to the correct value of 201402L (from 201300L). It occurs to me: instead of having to remember every time those numbers, couldn't we predefine, for example: __cplusplus_98

Re: [patch, fortran] use vec<> in frontend-passes.c

2014-08-26 Thread Tobias Burnus
Hi Thomas, hi all, first, thanks Trevor for the first round of review! Thomas Koenig wrote: > Trevor Saunders wrote: > > >doloop_warn (ns); > > > - XDELETEVEC (doloop_list); > > > + doloop_list.truncate (0); > > > > .release () would be more typical. > > Changed (also below). I still see:

Re: [Patch 1/2] Don't put out a call to memcpy for volatile struct operations

2014-08-26 Thread Richard Biener
On Fri, Aug 22, 2014 at 1:47 AM, Mike Stump wrote: > On Aug 21, 2014, at 4:22 AM, Richard Biener > wrote: >> I still say we need to solve the issue at language level - that is, >> try to figure out what the language standard says about >> >> volatile struct X x, y; > >> x = y; > > The definition

Re: [PATCH. libstdc++] Use the correct C++14 __cplusplus value (201402L). Added C++1z to the preprocessor.

2014-08-26 Thread Rainer Orth
Paolo Carlini writes: > Hi, > > On 08/24/2014 12:37 AM, Ed Smith-Rowland wrote: >> With revision 214400 we have the C++14 value of __cplusplus set to the >> correct value of 201402L (from 201300L). > It occurs to me: instead of having to remember every time those numbers, > couldn't we predefine,

Re: [PATCH] Fix LOOP_REQUIRES_VERSIONING_* macros (PR c/61271)

2014-08-26 Thread Richard Biener
On Mon, Aug 25, 2014 at 3:02 PM, Marek Polacek wrote: > Another issue found by -Wlogical-not-parentheses. > tree-vect-data-refs.c contains > if (!LOOP_REQUIRES_VERSIONING_FOR_ALIGNMENT (loop_vinfo)) > but since the definition of LOOP_REQUIRES_VERSIONING_FOR_ALIGNMENT > is lacking parens, this expa

Re: [PATCH GCC]Fix broken Canadian when checking isl library support

2014-08-26 Thread Richard Biener
On Mon, Aug 25, 2014 at 11:07 PM, Sebastian Pop wrote: > Sebastian Pop wrote: >> Richard Biener wrote: >> > I think it would be better to identify a set of features we rely on that >> > are not present in earlier versions and make the test a link >> > test unconditionally. >> > >> > Tobias, are th

RE: [PATCH][ARM] Fix -fcall-saved-rX for X > 7 When compiling for size for thumb targets

2014-08-26 Thread Thomas Preud'homme
Ping? > -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- > ow...@gcc.gnu.org] On Behalf Of Thomas Preud'homme > Sent: Wednesday, August 20, 2014 9:28 AM > To: gcc-patches@gcc.gnu.org > Subject: [PATCH][ARM] Fix -fcall-saved-rX for X > 7 > > This patch makes -f

[PING][PATCH] Fix for PR 61875

2014-08-26 Thread Marat Zakirov
On 08/18/2014 07:37 PM, Marat Zakirov wrote: Hi there! I have a fix for https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61875. This situation occurs when somebody decides to build GCC with -fexeptions and -frtti which are forbidden for libsanitizer. They get strange error (see bug above) which

Re: PATCH: PR bootstrap/62260: Build inside source tree doesn't work with lto-plugin

2014-08-26 Thread Richard Biener
On Mon, Aug 25, 2014 at 11:54 PM, H.J. Lu wrote: > Inside lto-plugin, when $host_subdir is '.', which means GCC is configured > inside source tree, gcc_build_dir is ../../$host_subdir/gcc, not > ../$(host_subdir)/gcc. Tested with both in-tree and out-tree build on > Linux/x86-64. OK for trunk? O

Re: [PING][PATCH] Fix for PR 61875

2014-08-26 Thread Yury Gribov
On 08/18/2014 07:37 PM, Marat Zakirov wrote: Hi there! I have a fix for https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61875. This situation occurs when somebody decides to build GCC with -fexeptions and -frtti which are forbidden for libsanitizer. They get strange error (see bug above) which I kn

Re: [PATCH. libstdc++] Use the correct C++14 __cplusplus value (201402L). Added C++1z to the preprocessor.

2014-08-26 Thread Jonathan Wakely
On 26/08/14 10:28 +0200, Paolo Carlini wrote: Hi, On 08/24/2014 12:37 AM, Ed Smith-Rowland wrote: With revision 214400 we have the C++14 value of __cplusplus set to the correct value of 201402L (from 201300L). It occurs to me: instead of having to remember every time those numbers, couldn't we

Re: [PATCH. libstdc++] Use the correct C++14 __cplusplus value (201402L). Added C++1z to the preprocessor.

2014-08-26 Thread Paolo Carlini
Hi, On 08/26/2014 10:42 AM, Rainer Orth wrote: Paolo Carlini writes: Hi, On 08/24/2014 12:37 AM, Ed Smith-Rowland wrote: With revision 214400 we have the C++14 value of __cplusplus set to the correct value of 201402L (from 201300L). It occurs to me: instead of having to remember every time

Re: [PATCH. libstdc++] Use the correct C++14 __cplusplus value (201402L). Added C++1z to the preprocessor.

2014-08-26 Thread Paolo Carlini
Hi, On 08/26/2014 10:52 AM, Jonathan Wakely wrote: That seems like a good idea, but I'm not convinced there's any benefit in Ed's changes to do: -#if __cplusplus > 201103L +#if __cplusplus >= 201402L It seems like unnecessary churn to me, about this, I had the same thought ;) Paolo.

Re: [PATCH. libstdc++] Use the correct C++14 __cplusplus value (201402L). Added C++1z to the preprocessor.

2014-08-26 Thread Rainer Orth
Hi Paolo, > On 08/26/2014 10:42 AM, Rainer Orth wrote: >> Paolo Carlini writes: >> >>> Hi, >>> >>> On 08/24/2014 12:37 AM, Ed Smith-Rowland wrote: With revision 214400 we have the C++14 value of __cplusplus set to the correct value of 201402L (from 201300L). >>> It occurs to me: instead

Re: Enable EBX for x86 in 32bits PIC code

2014-08-26 Thread Ilya Enkovich
2014-08-26 11:49 GMT+04:00 Ilya Enkovich : > 2014-08-25 19:08 GMT+04:00 Vladimir Makarov : >> On 2014-08-22 8:21 AM, Ilya Enkovich wrote: >>> >>> Hi, >>> >>> On Cauldron 2014 we had a couple of talks about relaxation of ebx usage in >>> 32bit PIC mode. It was decided that the best approach would b

Re: [PATCH GCC]Fix broken Canadian when checking isl library support

2014-08-26 Thread Bin.Cheng
On Tue, Aug 26, 2014 at 4:46 PM, Richard Biener wrote: > On Mon, Aug 25, 2014 at 11:07 PM, Sebastian Pop wrote: >> Sebastian Pop wrote: >>> Richard Biener wrote: >>> > I think it would be better to identify a set of features we rely on that >>> > are not present in earlier versions and make the t

Re: [PATCH. libstdc++] Use the correct C++14 __cplusplus value (201402L). Added C++1z to the preprocessor.

2014-08-26 Thread Paolo Carlini
Hi, On 08/26/2014 10:56 AM, Rainer Orth wrote: bits/c++config... Which won't help users seeing them in the headers. Maybe the issue could be avoided by chosing names that make it clear that they are g++/libstdc++ specific, not generic? Sure, whatever works, the names were tentative, for the cp

Re: [PATCH, Fortran] PR fortran/60414 fix ICE was: PR 60414: Patch proposal

2014-08-26 Thread Andre Vehreschild
Hi, On Sun, 17 Aug 2014 15:06:02 +0200 Mikael Morin wrote: > Le 17/08/2014 14:26, Dominique Dhumieres a écrit : > > As Mikael said in https://gcc.gnu.org/ml/fortran/2014-08/msg00047.html > > > >> the testcase should check that the code generated is actually working, > >> not just that the ICE d

[PATCH][match-and-simplify] Introduce (with { .. } op), combine conversions

2014-08-26 Thread Richard Biener
The following patch introduces a new syntax to initialize temporaries used in c-exprs (such as if conditionals). So a simplify pattern is now 'simplify' [ ] with = | | = '(' 'if' '(' ')' ')' = '(' 'with' '{' '}' ')' which allows nearly literal translation of

Re: [PATCH. libstdc++] Use the correct C++14 __cplusplus value (201402L). Added C++1z to the preprocessor.

2014-08-26 Thread Ed Smith-Rowland
On 08/26/2014 04:59 AM, Paolo Carlini wrote: Hi, On 08/26/2014 10:56 AM, Rainer Orth wrote: bits/c++config... Which won't help users seeing them in the headers. Maybe the issue could be avoided by chosing names that make it clear that they are g++/libstdc++ specific, not generic? Sure, whatev

Re: Make string_view operations involving CharT* *not* noexcept and consistent beween string_view and string_view.tcc.

2014-08-26 Thread Jonathan Wakely
On 15/04/14 20:06 +0100, Jonathan Wakely wrote: On 29/03/14 14:54 -0400, Ed Smith-Rowland wrote: All, In string_view I botched the noexcept specification of operations like find and friends with CharT* arguments. I'm a little surprised the inconsistency between string_view and string_view.t

Re: [PATCH. libstdc++] Use the correct C++14 __cplusplus value (201402L). Added C++1z to the preprocessor.

2014-08-26 Thread Jonathan Wakely
On 26/08/14 07:11 -0400, Ed Smith-Rowland wrote: On 08/26/2014 04:59 AM, Paolo Carlini wrote: Hi, On 08/26/2014 10:56 AM, Rainer Orth wrote: bits/c++config... Which won't help users seeing them in the headers. Maybe the issue could be avoided by chosing names that make it clear that they are

Re: [PATCH, AArch64] Fix typo

2014-08-26 Thread Richard Earnshaw
On 15/08/14 17:24, Evandro Menezes wrote: > I tripped at a typo that goes undetected because the macro NAMED_PARAM > doesn't apply in the absence of designated initializers. > > Since struct scale_addr_mode_cost has the cost for DI, but not for QI, the > instances of struct cpu_addrcost_table are

[patch] Update C++11 library implementation status for 4.8 branch

2014-08-26 Thread Jonathan Wakely
Committed to the 4.8 branch. commit d926658a428654221ecbf42f431afa43abe4ac00 Author: Jonathan Wakely Date: Tue Aug 26 13:15:58 2014 +0100 * doc/xml/manual/status_cxx2011.xml: Correct status table. * doc/html/manual/*: Regenerate. diff --git a/libstdc++-v3/doc/html/manual/abi.html b/

Re: [PATCH][ARM] Optimize copysign/copysignf for soft-float using BFI

2014-08-26 Thread Richard Earnshaw
On 29/07/14 15:49, Jiong Wang wrote: > this patch optimize copysign/copysignf for -mfloat-abi=soft on arm when BFI > instruction is available. > > before this patch, we do copysign (a, b) by three steps: >* fetch the sign bit of b to A >* fetch all non-sign bits of a to B >* or A and

Re: [RTL, i386] Use subreg instead of UNSPEC_CAST

2014-08-26 Thread Marc Glisse
Ping? On Sat, 26 Jul 2014, Marc Glisse wrote: Hello, any comment on this patch? https://gcc.gnu.org/ml/gcc-patches/2014-06/msg00769.html On Tue, 10 Jun 2014, Marc Glisse wrote: On Tue, 19 Mar 2013, Richard Henderson wrote: I'm not fond of this, primarily because I believe the pattern shou

Re: [PATCH 3/4] aarch64: Tidy prologue local variables

2014-08-26 Thread Jiong Wang
On 22/08/14 23:05, Richard Henderson wrote: Don't continually re-read data from cfun->machine. * config/aarch64/aarch64.c (aarch64_expand_prologue): Load cfun->machine->frame.hard_fp_offset into a local variable. --- gcc/config/aarch64/aarch64.c | 14 +++--- 1 file cha

Re: [PATCH 1/2, x86] Add palignr support for AVX2.

2014-08-26 Thread Evgeny Stupachenko
That is covered by a separate part of the patch: (make check and bootstrap passed: 2 new passes for core-avx2) is it ok? diff --git a/gcc/config/i386/sse.md b/gcc/config/i386/sse.md index d6155cf..68ee65a 100644 --- a/gcc/config/i386/sse.md +++ b/gcc/config/i386/sse.md @@ -81,6 +81,7 @@ ;; For

[PATCH][match-and-simplify] Sort IVOPTs iv_uses after dominator

2014-08-26 Thread Richard Biener
The following is needed to allow building libada if the conversion simplifications go in. Currently IVOPTs replaces IV uses in arbitrary order which can result in intermediate code that still refers to IVs that are going to be removed. As it folds replacement statements via force_gimple_operand

Re: [PATCH]gfortran.dg/bessel_7.f90: Bump allowed precision to avoid failure on s390.

2014-08-26 Thread Tobias Burnus
Dominik Vogt wrote: > The attached patch fixes a bessel_7 fortran test failure that appeared during > SLES 12 distro tests for s390x - by bumping the error tolerance. > > The same patch should apply to gcc-4.9 and gcc-4.8. OK. Thanks for the patch. > Please do not commit this patch; one of my c

Re: [PATCH 1/4] aarch64: Improve epilogue unwind info

2014-08-26 Thread Jiong Wang
thanks, verified no regression on aarch64-none-elf bare-metal check-gcc/check-gdb. -- Jiong On 22/08/14 23:05, Richard Henderson wrote: Delay cfi restore opcodes until the stack frame is deallocated. This reduces the number of cfi advance opcodes required. We perform a similar optimization in

[PATCH][match-and-simplify] Fix some C++ testsuite FAILs

2014-08-26 Thread Richard Biener
The following fixes C++ testsuite FAILs. First in fold_binary we should delay the dispatch to generic_simplify until after const OP const folding which generic_simplify isn't supposed to do. (I note that neither fold_unary nor fold_ternary has such clear const operand path...) Second, the C++ F

Re: [PATCH libstdc++ v3] - Add xmethods for std::vector and std::unique_ptr

2014-08-26 Thread Siva Chandra
Friendly ping. On Wed, Aug 13, 2014 at 7:50 AM, Jonathan Wakely wrote: > On 13 August 2014 14:55, Siva Chandra wrote: >> Ping. Any update on this? > > No, I haven't committed it yet, but I will get round to it. > >> On Wed, Aug 6, 2014 at 6:37 AM, Siva Chandra wrote: >>> On Wed, Aug 6, 2014 at 2

[PATCH] Fix PR62239

2014-08-26 Thread Richard Biener
The following fixes PR62239 and moves strcat_chk folding to GIMPLE. Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Richard. 2014-08-26 Richard Biener PR middle-end/62239 * builtins.c (fold_builtin_strcat_chk): Move to gimple-fold.c. (fold_builtin_3):

[PATCH] Fix -Wlogical-not-parentheses warning in sel-sched-ir.c (PR c/61271)

2014-08-26 Thread Marek Polacek
Another wrongdoing detected by -Wlogical-not-parentheses. From my reading of the code it seems that simple "!=" was meant here. Bootstrapped/regtested on x86_64-linux, ok for trunk? 2014-08-26 Marek Polacek PR c/61271 * sel-sched-ir.c (make_regions_from_the_rest): Fix conditi

Re: [PATCH] Fix -Wlogical-not-parentheses warning in sel-sched-ir.c (PR c/61271)

2014-08-26 Thread Andrey Belevantsev
On 26.08.2014 18:03, Marek Polacek wrote: Another wrongdoing detected by -Wlogical-not-parentheses. From my reading of the code it seems that simple "!=" was meant here. Bootstrapped/regtested on x86_64-linux, ok for trunk? OK. Heck, we traced this code back to the May 2007 commit :) Andrey

Re: [PATCH] Fix -Wlogical-not-parentheses warning in sel-sched-ir.c (PR c/61271)

2014-08-26 Thread Richard Biener
On Tue, Aug 26, 2014 at 4:16 PM, Andrey Belevantsev wrote: > On 26.08.2014 18:03, Marek Polacek wrote: >> >> Another wrongdoing detected by -Wlogical-not-parentheses. From >> my reading of the code it seems that simple "!=" was meant here. >> >> Bootstrapped/regtested on x86_64-linux, ok for trun

Re: [patch] Overload std::getline for rvalue streams

2014-08-26 Thread Jonathan Wakely
On 12/08/14 16:20 +0100, Jonathan Wakely wrote: We're missing these overloads required by C++11. Tested x86_64-linux, committed to trunk. I think this should go on the branches too, any objections? Committed to the 4.9 branch.

Re: [PATCH 1/2] Fix __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__

2014-08-26 Thread Mike Stump
On Aug 25, 2014, at 11:04 PM, James Clarke wrote: > So I should just take that particular check out then? Oh, yeah, I guess the is the way to fix it.

Re: Enable EBX for x86 in 32bits PIC code

2014-08-26 Thread Vladimir Makarov
On 08/26/2014 04:57 AM, Ilya Enkovich wrote: > 2014-08-26 11:49 GMT+04:00 Ilya Enkovich : >> 2014-08-25 19:08 GMT+04:00 Vladimir Makarov : >>> On 2014-08-22 8:21 AM, Ilya Enkovich wrote: Hi, On Cauldron 2014 we had a couple of talks about relaxation of ebx usage in 32bit PIC mod

Re: PATCH: PR bootstrap/62260: Build inside source tree doesn't work with lto-plugin

2014-08-26 Thread Jeff Law
On 08/26/14 02:48, Richard Biener wrote: On Mon, Aug 25, 2014 at 11:54 PM, H.J. Lu wrote: Inside lto-plugin, when $host_subdir is '.', which means GCC is configured inside source tree, gcc_build_dir is ../../$host_subdir/gcc, not ../$(host_subdir)/gcc. Tested with both in-tree and out-tree buil

[PATCH 0/3] Updated patches to eliminate need for rtx_expr_list::insn (was Re: [PATCH 221/236] Add insn method to rtx_expr_list)

2014-08-26 Thread David Malcolm
On Mon, 2014-08-25 at 08:22 -0600, Jeff Law wrote: On 08/19/14 14:41, Richard Henderson wrote: > > On 08/06/2014 10:23 AM, David Malcolm wrote: > >> gcc/ > >> * rtl.h (rtx_expr_list::insn): New method. > >> --- > >> gcc/rtl.h | 9 + > >> 1 file changed, 9 insertions(+) > >> > >> dif

[PATCH 1/3] Convert nonlocal_goto_handler_labels from an EXPR_LIST to an INSN_LIST

2014-08-26 Thread David Malcolm
gcc/ * function.h (struct rtl_data): Convert field "x_nonlocal_goto_handler_labels" from rtx_expr_list * to rtx_insn_list *. * rtl.h (remove_node_from_insn_list): New prototype. * builtins.c (expand_builtin): When prepending to nonlocal_goto_handler_

[PATCH 2/3] Convert forced_labels from an EXPR_LIST to an INSN_LIST

2014-08-26 Thread David Malcolm
gcc/ * function.h (struct expr_status): Convert field "x_forced_labels" from rtx_expr_list * to rtx_insn_list *. * cfgbuild.c (make_edges): Convert local "x" from an rtx_expr_list * to an rtx_insn_list *, replacing use of "element" method with "insn" method.

[PATCH 3/3] Use rtx_insn in more places in dwarf2cfi.c

2014-08-26 Thread David Malcolm
gcc/ * dwarf2cfi.c (dw_trace_info): Strengthen field "head" from rtx to rtx_insn *. (get_trace_info): Likewise for param "insn". (save_point_p): Likewise. (maybe_record_trace_start): Likewise for both params. (maybe_record_trace_start_abnormal): Likew

[PATCH v2 0/2] Fix OS X Yosemite-related issues

2014-08-26 Thread James Clarke
These patches combine to address PR target/61407, meaning that GCC is able to bootstrap on OS X 10.10 Yosemite. James Clarke (2): Fix __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__ Fixed dirent sanitizer for 32-bit OS X builds gcc/config/darwin-c.c | 25 ++

[PATCH v2 1/2] Fix __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__

2014-08-26 Thread James Clarke
Previously, this macro had taken the form 10X0, where X is the minor version number, e.g. 1090 for OS X 10.9 Mavericks. However, as of OS X 10.10 Yosemite, it should take the form 10XX00, i.e. 101000 for 10.10. Added a test for the new format and fixed the formatting for the existing ones. gcc/Ch

[PATCH v2 2/2] Fixed dirent sanitizer for 32-bit OS X builds

2014-08-26 Thread James Clarke
The __DARWIN_64_BIT_INO_T macro is used to indicate whether dirent.d_ino is 64-bit or not, and so should be sized appropriately. Additionally, dirent.d_seekoff should only be defined/checked when __DARWIN_64_BIT_INO_T is defined. libsanitizer/ChangeLog: * sanitizer_common/sanitizer_platform_l

Re: [PATCH 1/2, x86] Add palignr support for AVX2.

2014-08-26 Thread Richard Henderson
On 08/26/2014 05:59 AM, Evgeny Stupachenko wrote: > +(define_insn_and_split "avx2_rotate_perm" > + [(set (match_operand:V_256 0 "register_operand" "=&x") > + (vec_select:V_256 > + (match_operand:V_256 1 "register_operand" "x") > + (match_parallel 2 "palignr_operand" > + [(

Re: [PATCH 0/3] Updated patches to eliminate need for rtx_expr_list::insn (was Re: [PATCH 221/236] Add insn method to rtx_expr_list)

2014-08-26 Thread Richard Henderson
On 08/26/2014 09:00 AM, David Malcolm wrote: > OK for trunk? > > David Malcolm (3): > Convert nonlocal_goto_handler_labels from an EXPR_LIST to an INSN_LIST > Convert forced_labels from an EXPR_LIST to an INSN_LIST > Use rtx_insn in more places in dwarf2cfi.c Ok to all. Thanks. r~

Re: [Patch 1/2] Don't put out a call to memcpy for volatile struct operations

2014-08-26 Thread Mike Stump
On Aug 26, 2014, at 1:35 AM, Richard Biener wrote: > >> 8 The implicitly-defined copy constructor for class X performs a member- >> wise copy of its subobjects. The order of copying is the same as the >> order of initialization of bases and members in a user-defined con- >> structor (see

Re: [PATCH v2 0/2] Fix OS X Yosemite-related issues

2014-08-26 Thread Mike Stump
On Aug 26, 2014, at 9:02 AM, James Clarke wrote: > These patches combine to address PR target/61407, meaning that GCC is > able to bootstrap on OS X 10.10 Yosemite. > James Clarke (2): > Fix __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__ For the first patch, Ok. So, after baking for a week, I t

Re: [PATCH 225/236] Work towards NEXT_INSN/PREV_INSN requiring insns as their params

2014-08-26 Thread David Malcolm
On Mon, 2014-08-25 at 08:25 -0600, Jeff Law wrote: > On 08/19/14 15:35, David Malcolm wrote: > > On Tue, 2014-08-19 at 13:57 -0700, Richard Henderson wrote: > >> On 08/06/2014 10:23 AM, David Malcolm wrote: > >>> diff --git a/gcc/cfgrtl.c b/gcc/cfgrtl.c > >>> index 59d633d..5e42a97 100644 > >>> ---

[PATCH v3 0/2] Fix OS X Yosemite-related issues

2014-08-26 Thread James Clarke
These patches combine to address PR target/61407, meaning that GCC is able to bootstrap on OS X Yosemite. Changes from v2: [PATCH v3 1/2] Fix __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__: * Unchanged [PATCH v3 2/2] Fixed dirent sanitizer for 32-bit OS X builds: * Switch to using the p

[PATCH v3 1/2] Fix __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__

2014-08-26 Thread James Clarke
Previously, this macro had taken the form 10X0, where X is the minor version number, e.g. 1090 for OS X 10.9 Mavericks. However, as of OS X 10.10 Yosemite, it should take the form 10XX00, i.e. 101000 for 10.10. Added a test for the new format and fixed the formatting for the existing ones. gcc/Ch

[PATCH v3 2/2] Fixed dirent sanitizer for 32-bit OS X builds

2014-08-26 Thread James Clarke
The _DARWIN_FEATURE_64_BIT_INODE macro indicates whether dirent.d_ino is 64-bit or not, which should be sized appropriately, and whether dirent.d_seekoff should be defined/checked. libsanitizer/ChangeLog: * sanitizer_common/sanitizer_platform_limits_posix.cc: Only check dirent.d_seekoff o

Re: [PATCH libstdc++ v3] - Add xmethods for std::vector and std::unique_ptr

2014-08-26 Thread Jonathan Wakely
Shouldn't there be a change to python/Makefile.am so that xmethods.py gets installed alongside printers.py? Otherwise you can use these new xmethods in the libstdc++ testssuite, but they're not available to users. I'd also expect something to call the register_libstdcxx_xmethods function automati

[PATCH v4 0/2] Fix OS X Yosemite-related issues

2014-08-26 Thread James Clarke
These patches combine to address PR target/61407, meaning that GCC is able to bootstrap on OS X Yosemite. Changes from v3: [PATCH v4 1/2] Fix __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__: * Unchanged [PATCH v4 2/2] Fixed dirent sanitizer for 32-bit OS X builds: * Check whether the _DA

[PATCH v4 1/2] Fix __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__

2014-08-26 Thread James Clarke
Previously, this macro had taken the form 10X0, where X is the minor version number, e.g. 1090 for OS X 10.9 Mavericks. However, as of OS X 10.10 Yosemite, it should take the form 10XX00, i.e. 101000 for 10.10. Added a test for the new format and fixed the formatting for the existing ones. gcc/Ch

[PATCH v4 2/2] Fixed dirent sanitizer for 32-bit OS X builds

2014-08-26 Thread James Clarke
The _DARWIN_FEATURE_64_BIT_INODE macro indicates whether dirent.d_ino is 64-bit or not, which should be sized appropriately, and whether dirent.d_seekoff should be defined/checked. libsanitizer/ChangeLog: * sanitizer_common/sanitizer_platform_limits_posix.cc: Only check dirent.d_seekoff o

Re: [patch, fortran] use vec<> in frontend-passes.c

2014-08-26 Thread Thomas Koenig
Hi Tobias, > first, thanks Trevor for the first round of review! Also thanks from my side! > I still see: >> - expr_count = 0; >> + expr_array.truncate (0); > > Is there is a reason for not using release() here? No, changed in the committed version. Regards Thomas

Re: [C++ RFH/Patch] PR 52892 (and others)

2014-08-26 Thread Jason Merrill
On 08/26/2014 12:01 PM, Paolo Carlini wrote: the difference, for the latter and for more complex cases, is that adjust_temp_type calls cp_fold_convert which ends up returning a NOP_EXPR (eg, build in fold_convert_loc). Perhaps we should address this in adjust_temp_type, either by ignoring the

Re: [PATCH 40/50] rtlanal.c:for_each_inc_dec

2014-08-26 Thread Richard Sandiford
[Jeff, sorry for the duplicate, sent the original from an account that adds disclaimers.] Jeff Law writes: > On 08/09/14 04:13, Richard Sandiford wrote: >> Jeff Law writes: >>> On 08/03/14 08:32, Richard Sandiford wrote: The old for_each_inc_dec callback had a for_each_rtx-like return value

C++ PATCH for middle-end/58624 (thread_local static data member)

2014-08-26 Thread Jason Merrill
This PR points out an ICE caused by thread_local on static data members, which we weren't handling properly at all. There is still the issue that calling set_decl_tls_model on a template variable adds it to the symbol table, causing problems later on, but this patch fixes the front end handlin

Re: [C++ RFH/Patch] PR 52892 (and others)

2014-08-26 Thread Paolo Carlini
Hi, On 08/26/2014 08:58 PM, Jason Merrill wrote: On 08/26/2014 12:01 PM, Paolo Carlini wrote: the difference, for the latter and for more complex cases, is that adjust_temp_type calls cp_fold_convert which ends up returning a NOP_EXPR (eg, build in fold_convert_loc). Perhaps we should address

Re: [PATCH i386 AVX512] [22/n] Extend unaligned loads & stores.

2014-08-26 Thread Kirill Yukhin
Hello Uroš, On 23 Aug 09:44, Uros Bizjak wrote: > On Fri, Aug 22, 2014 at 1:51 PM, Kirill Yukhin > wrote: > > > This patch extends unaligned loads and stores patterns. > At this stage, I'd still prefer simple constraints (the solution, > proposed above), even for the price of additional patterns

[patch] Why xstrdup cgraph node names for dumpfiles?

2014-08-26 Thread Steven Bosscher
Hello Martin, Honza, I noticed most of the cgraph and IPA files use xstrdup for cgraph node names when printing to dump_file. Very leaky... What is the reason for all those xstrdups? I couldn't think of any. Thoughts? Ciao! Steven * cgraph.c (cgraph_node::get_create): Don't xstrdup cgr

Fwd: [PATCH i386 AVX512] [22/n] Extend unaligned loads & stores.

2014-08-26 Thread Uros Bizjak
On Tue, Aug 26, 2014 at 9:55 PM, Kirill Yukhin wrote: > Hello Uroš, > On 23 Aug 09:44, Uros Bizjak wrote: >> On Fri, Aug 22, 2014 at 1:51 PM, Kirill Yukhin >> wrote: >> >> > This patch extends unaligned loads and stores patterns. >> At this stage, I'd still prefer simple constraints (the solutio

Re: [patch] Why xstrdup cgraph node names for dumpfiles?

2014-08-26 Thread Uros Bizjak
Hello! > I noticed most of the cgraph and IPA files use xstrdup for cgraph node > names when printing to dump_file. Very leaky... > > What is the reason for all those xstrdups? I couldn't think of any. Please see [1] and [2]. [1] https://gcc.gnu.org/ml/gcc-patches/2012-04/msg01904.html [2] https

Re: [patch] Why xstrdup cgraph node names for dumpfiles?

2014-08-26 Thread Steven Bosscher
On Tue, Aug 26, 2014 at 10:52 PM, Uros Bizjak wrote: > Hello! > >> I noticed most of the cgraph and IPA files use xstrdup for cgraph node >> names when printing to dump_file. Very leaky... >> >> What is the reason for all those xstrdups? I couldn't think of any. > > Please see [1] and [2]. > > [1]

[patch] make dwarf2out_decl static

2014-08-26 Thread Aldy Hernandez
This doesn't seem to be used anywhere but in dwarf2out.c. OK for mainline? commit 8a48cdd576442cee103399ab4f1f67ee215ee6b8 Author: Aldy Hernandez Date: Tue Aug 26 14:30:22 2014 -0700 * dwarf2out.h (dwarf2out_decl): Remove prototype. * dwarf2out.c (dwarf2out_decl): Make static.

Re: [patch] make dwarf2out_decl static

2014-08-26 Thread Jeff Law
On 08/26/14 15:32, Aldy Hernandez wrote: This doesn't seem to be used anywhere but in dwarf2out.c. OK for mainline? OK jeff

Re: Enable EBX for x86 in 32bits PIC code

2014-08-26 Thread Ilya Enkovich
On 26 Aug 11:25, Vladimir Makarov wrote: > On 08/26/2014 04:57 AM, Ilya Enkovich wrote: > > I've looked into one of fails. There is still a problem with > > allocation in reload. Here is a piece of code which uses float > > constant: > > > > (insn 1199 1198 1200 96 (set (reg:SI 3 bx) > > (

[Patch AArch64] Fix for PR62262

2014-08-26 Thread Carrot Wei
Hi In insn pattern "*andim_ashift_bfiz", if the operands[2] is larger than the size of register, gcc may generate invalid assembler code. If operands[2] is larger than the size of the underlying type of INTVAL, the following insn condition may also be undefined. "exact_log2 ((INTVAL (operands[3])

Re: Doc Bug: cxa-atexit not use-cxa-atexit

2014-08-26 Thread Joel Sherrill
On 8/23/2014 11:19 AM, Gerald Pfeifer wrote: > On Mon, 18 Aug 2014, Joel Sherrill wrote: >> I think this is a minor documentation bug which is in the head but also >> seems to be in the gcc 4.4.7 docs shipped with CentOS 6.x. >> >> OK to commit? >> >> 2014-08-18 Joel Sherrill >> >> * doc/inv

[committed] Update 4.8 baseline symbols on hppa-linux

2014-08-26 Thread John David Anglin
Attached is an update to the baseline symbols on hppa-linux to fix the abi test failure in the current 4.8 tree. Dave -- John David Anglin dave.ang...@bell.net 2014-08-26 John David Anglin * config/abi/post/hppa-linux-gnu/baseline_symbols.txt: Update. Index: config/abi/post

[PATCH,rs6000] Add some more vector built-ins

2014-08-26 Thread Bill Schmidt
Hi, This patch adds a few more cases of overloaded vector built-ins to support V2DI and V2DF modes: vec_xl, vec_xst, vec_splat, vec_div, vec_mul, vec_round. These are all straightforward. For vec_div and vec_mul, the most efficient thing appears to be to just scalarize these; at least I couldn'

[debug-early] reuse variable DIEs and fix their context

2014-08-26 Thread Aldy Hernandez
This patch fixes a bunch of guality failures. With it I get 144 guality.exp failures vs. 163 for "make check-gcc RUNTESTFLAGS=guality.exp". A lot better than 100% fail rate ;-). Variable DIEs were not being reused. Instead, variable DIEs even had the wrong context (unilaterally the compilat

Go patch committed: Disallow call of *T method using **T variable

2014-08-26 Thread Ian Lance Taylor
This patch by Chris Manghane fixes the Go frontend to not permit calling a pointer method on *T with a variable of **T. Previously it was incorrectly dereferencing the **T to get a *T and using that to call the method. Fixing this required tweaking one test case. Bootstrapped and ran Go testsuit

Re: [PATCH 4.8] libstdc++ pretty-printers: Backport Python 3 support from mainline

2014-08-26 Thread Samuel Bronson
Ping! -- Hi! I'm a .signature virus! Copy me into your ~/.signature to help me spread!