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 richard.guent...@gmail.com wrote: On Mon, Aug 25, 2014 at 11:07 PM, Sebastian Pop seb...@gmail.com 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

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

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 mikael.mo...@sfr.fr 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

[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' [ ident ] expr result-op with result-op = op | if | with if = '(' 'if' '(' c-expr ')' result-op ')' with = '(' 'with'

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,

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

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 not

[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 jwak...@redhat.com Date: Tue Aug 26 13:15:58 2014 +0100 * doc/xml/manual/status_cxx2011.xml: Correct status table. * doc/html/manual/*: Regenerate. diff --git

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 B

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

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

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

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

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

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 jwakely@gmail.com 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 sivachan...@google.com

[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 rguent...@suse.de PR middle-end/62239 * builtins.c (fold_builtin_strcat_chk): Move to gimple-fold.c.

[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 pola...@redhat.com PR c/61271 * sel-sched-ir.c

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 a...@ispras.ru 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

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 jrt...@jrtc27.com 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 enkovich@gmail.com: 2014-08-25 19:08 GMT+04:00 Vladimir Makarov vmaka...@redhat.com: On 2014-08-22 8:21 AM, Ilya Enkovich wrote: Hi, On Cauldron 2014 we had a couple of talks about relaxation of ebx

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 hongjiu...@intel.com 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

[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(+) diff --git a/gcc/rtl.h

[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

[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):

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

[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: *

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_rotatemode_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 + [(match_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 richard.guent...@gmail.com 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

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 jrt...@jrtc27.com 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

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 --- a/gcc/cfgrtl.c

[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

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

[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

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

[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

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

[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

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 l...@redhat.com writes: On 08/09/14 04:13, Richard Sandiford wrote: Jeff Law l...@redhat.com writes: On 08/03/14 08:32, Richard Sandiford wrote: The old for_each_inc_dec callback had a

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

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

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

[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

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 kirill.yuk...@gmail.com wrote: Hello Uroš, On 23 Aug 09:44, Uros Bizjak wrote: On Fri, Aug 22, 2014 at 1:51 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: This patch extends unaligned loads and stores patterns. At this stage, I'd still

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]

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 al...@redhat.com Date: Tue Aug 26 14:30:22 2014 -0700 * dwarf2out.h (dwarf2out_decl): Remove prototype. * dwarf2out.c

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) (reg:SI 1301

[Patch AArch64] Fix for PR62262

2014-08-26 Thread Carrot Wei
Hi In insn pattern *andim_ashiftmode_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

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 joel.sherr...@oarcorp.com

[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 dang...@gcc.gnu.org * config/abi/post/hppa-linux-gnu/baseline_symbols.txt: Update.

[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

[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

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

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!

Re: [PATCH 1/2] Fix __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__

2014-08-26 Thread James Clarke
So I should just take that particular check out then? James Clarke On 25 Aug 2014, at 21:05, Mike Stump mikest...@comcast.net wrote: On Aug 25, 2014, at 11:03 AM, James Clarke jrt...@jrtc27.com wrote: Previously, this macro had taken the form 10X0, where X is the minor version number, e.g.

[PATCH] Small fix for r214086 Asan regression

2014-08-26 Thread Yury Gribov
Hi all, Deja pattern in r214086's test is too strict (Asan runtime will not always be able to detect that invalid memory access is use-after-poison). I've cooked a trivial patch to loosen the check so test now also works on i386. Tested on x64 and i386. Ok to commit? -Y commit

Re: [PATH] Intel offload library

2014-08-26 Thread Andrey Turetskiy
Ping. On Tue, Aug 19, 2014 at 12:45 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello, On 12 Aug 10:58, Andrey Turetskiy wrote: All remarks from https://gcc.gnu.org/ml/gcc-patches/2014-06/msg02166.html are fixed. Updated version of liboffloadmic you can find in GIT branch

[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 to rtx_insn *. *

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

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 rguent...@suse.de * 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 pola...@redhat.com 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

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: - expr_count

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 mikest...@comcast.net wrote: On Aug 21, 2014, at 4:22 AM, Richard Biener richard.guent...@gmail.com 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

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 paolo.carl...@oracle.com 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,

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

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

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

[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 hongjiu...@intel.com 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.

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

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

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 paolo.carl...@oracle.com 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

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 paolo.carl...@oracle.com 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 enkovich@gmail.com: 2014-08-25 19:08 GMT+04:00 Vladimir Makarov vmaka...@redhat.com: 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