libgo patch committed: Update to final 1.6 release

The Go 1.6 release went out earlier today. This patch updates libgo to the final version of the Go 1.6 libraries. Bootstrapped and tested on x86_64-pc-linux-gnu. Committed to mainline. Ian patch.txt.gz Description: GNU Zip compressed data

RE: RFC: [Patch, PR Bug 60818] - ICE in validate_condition_mode on powerpc*-linux-gnu* ]

> -Original Message- > From: Alan Modra [mailto:amo...@gmail.com] > On Wed, Feb 17, 2016 at 06:31:45AM -0600, Segher Boessenkool wrote: > > > (gdb) p debug_rtx (other_insn) > > > (insn 11 10 16 2 (set (reg:SI 165 [ D.2339+-3 ]) > > > (if_then_else:SI (ne (reg:CC 166) > > >

C++ PATCH for c++/68679 (ICE with member template in unnamed struct)

...because apparently someone wants to write such a thing? Anyway, it's a simple matter of looking through templates when calling reset_decl_linkage. Tested x86_64-pc-linux-gnu, applying to trunk and 5. commit c96622dda7826a32ccc3caa34d727d8c1e7c3b3f Author: Jason Merrill Date: Wed Feb 17 1

C++ PATCH for c++/68585 (wrong error with constexpr list-initialization)

init_subob_ctx changed value, and then evaluating value didn't change it, so we were thinking nothing had changed. Fixed by remembering the value from before init_subob_ctx. Tested x86_64-pc-linux-gnu, applying to trunk and 5. commit afc049e2b450d710e62e3c0112e30a9600d6 Author: Jason Merri

C++ PATCH for c++/65985 (wrong error with constexpr ctor)

The declaration of __PRETTY_FUNCTION__ was confusing build_constexpr_constructor_member_initializers. Tested x86_64-pc-linux-gnu, applying to trunk. commit aef1a26de333aab5f602f6eb02cfd1ae751bb84f Author: Jason Merrill Date: Wed Feb 17 16:33:50 2016 -0500 PR c++/65985 * constexpr.c

Re: [PATCH] 69780 - [4.9/5/6 Regression] ICE on __builtin_alloca_with_align, with small alignment

On 02/17/2016 03:50 PM, Jakub Jelinek wrote: On Wed, Feb 17, 2016 at 01:48:26PM -0700, Martin Sebor wrote: I shifted the alignment so that it could be compared against MAX_STACK_ALIGNMENT. But after some searching it seems as though MAX_STACK_ALIGNMENT is in bits, rather than bytes as I had ass

[wwwdocs] Changes for LTO and IPA, ver 2

Hi, it seems I have updated the patch for comments received but did not send updated version to the ML. Here it is. Honza Index: changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-6/changes.html,v retrieving revision 1.58 diff -c

Re: [PATCH] Fix PR c++/68948 (wrong code generation due to invalid constructor call)

OK. Jason

Re: PPC libgcc IEEE128 soft-fp exception/rounding fixes

On Wed, Feb 17, 2016 at 05:40:01PM -0600, Paul E. Murphy wrote: > - FP_INIT_ROUNDMODE writes junk to the fpscr. I assume this should be > reading the fpscr and initializing the local rounding mode variable > declared via _FP_DECL_EX. Yeah, looks that way. > - FP_TRAPPING_EXCEPTIONS evaluates

Re: PPC libgcc IEEE128 soft-fp exception/rounding fixes

I have no comments on this patch beyond again suggesting that it would be desirable for some future float128 GCC patch to enable as many as possible of the x86 float128 tests (that is, tests of float128 anywhere in the GCC testsuite that currently are restricted to x86, x86_64 and ia64 targets o

PPC libgcc IEEE128 soft-fp exception/rounding fixes

Hi all, I am fairly new to IBM and recently appointed maintainer of libdfp, and work on glibc on ppc. This is my first foray into libgcc. libdfp implements many common transcendental functions, overrides type conversions between decimal float and other GCC types with more optimized variants for d

Re: RFC: [Patch, PR Bug 60818] - ICE in validate_condition_mode on powerpc*-linux-gnu* ]

On Wed, Feb 17, 2016 at 06:31:45AM -0600, Segher Boessenkool wrote: > > Corresponding content of "op" which causes the ICE: > > gdb) p debug_rtx (op) > > (gtu:SI (reg:CC 166) -- (operator and mode doesn't > > match) > > (const_int 0 [0])) > > That is invalid RTL for this

Re: [PATCH] 69780 - [4.9/5/6 Regression] ICE on __builtin_alloca_with_align, with small alignment

On Wed, Feb 17, 2016 at 01:48:26PM -0700, Martin Sebor wrote: > I shifted the alignment so that it could be compared against > MAX_STACK_ALIGNMENT. But after some searching it seems as though > MAX_STACK_ALIGNMENT is in bits, rather than bytes as I had assumed, > so I've removed the shift. The re

Re: [PATCH] 69780 - [4.9/5/6 Regression] ICE on __builtin_alloca_with_align, with small alignment

On 02/17/2016 12:39 AM, Jakub Jelinek wrote: On Tue, Feb 16, 2016 at 06:04:48PM -0700, Martin Sebor wrote: Formatting. = needs to be on the next line. There are literally dozens of examples of this style in this file alone. In one of the two instances of this style in this patch, moving the

C++ PATCH for c++/69842 (wrong error with generic lambda)

The problem here was that the call from the stub returned by the conversion function to the op() was changing an xvalue to an lvalue, leading to a parameter of the wrong type in the op(). Tested x86_64-pc-linux-gnu, applying to trunk. commit 54448906d76dd9df2523e6c3950ed368cb63b4de Author: Jaso

Re: [PATCH] xtensa: fix libgcc build with --text-section-literals

On Wed, Feb 17, 2016 at 10:59 PM, augustine.sterl...@gmail.com wrote: > On Tue, Feb 16, 2016 at 4:35 PM, Max Filippov wrote: >> Functions __muldf3_aux, __divdf3_aux, __mulsf3_aux and __divsf3_aux >> don't start with leaf_entry, so they need explicit .literal_position, >> otherwise libgcc build fa

Re: [PATCH] xtensa: fix libgcc build with --text-section-literals

On Tue, Feb 16, 2016 at 4:35 PM, Max Filippov wrote: > Functions __muldf3_aux, __divdf3_aux, __mulsf3_aux and __divsf3_aux > don't start with leaf_entry, so they need explicit .literal_position, > otherwise libgcc build fails in the presence of --text-section-literals. > > 2016-02-17 Max Filippov

Re: [PATCH] Fix a C++ -Wnonnull-compare regression (PR c++/69850)

OK. Jason

[PATCH 9/9] S/390: z13 Add missing commutative operand markers.

gcc/ChangeLog: 2016-02-17 Andreas Krebbel * config/s390/vector.md: Add missing commutative operand markers to the patterns which qualify for one. * config/s390/vx-builtins.md: Likewise. --- gcc/config/s390/vector.md | 44 +-

[PATCH 1/9] S/390: Add IBM z13 pipeline description

This patch adds proper support for the -mtune=z13 option by adding a z13 pipeline description. As started with zEC12 we mostly make use of the sched reorder hooks to implement a grouping strategy. However, this time we also keep an eye at the instruction mix provided in the out of order window to

[PATCH 0/9] S/390: z13 pipeline description, stpcpy + bugfixes

I'm having this patchset in my local tree for quite a while now. Posting it was so far prevented by some internal process hurdles. I'm aware it isn't stage 4 material. I nevertheless would like to commit this since: * It is z13 only and z13 support was new in GCC 6 anyway. The risk to cause r

[PATCH 7/9] S/390: z13 Change predicates of 128 bit add sub.

So far usage of 128 bit add/sub instruction was rejected if the second operand was a constant because the predicate rejected this. gcc/testsuite/ChangeLog: 2016-02-17 Andreas Krebbel * gcc.target/s390/vector/int128-1.c: New test. gcc/ChangeLog: 2016-02-17 Andreas Krebbel

[PATCH 5/9] S/390: z13 fix mode in vcond expansion

For floating point vector compares the target mode is an integer mode which accidently was used as register mode when forcing the compare operands into regs. gcc/ChangeLog: 2016-02-17 Andreas Krebbel * config/s390/s390.c (s390_expand_vcond): Use the compare operand mode. gcc/

[PATCH 8/9] S/390: Add single element vector types to iterators.

gcc/ChangeLog: 2016-02-17 Andreas Krebbel * config/s390/vector.md (VI, VI_QHS): Add single element vector types to mode iterators. (vec_double): ... and mode attribute. * config/s390/vx-builtins.md (non_vec_int): Likewise. --- gcc/config/s390/vector.md | 1

[PATCH 3/9] S/390: z13 inline stpcpy implementation.

A handwritten loop for stpcpy using the new z13 vector instructions appears to be much faster than the millicoded instruction. However, the implementation is much longer and therefore will only be enabled when optimization for speed. gcc/testsuite/ChangeLog: 2016-02-17 Andreas Krebbel

[PATCH 6/9] S/390: Add vec_sub_u128 to vecintrin.h

This adds a missing macro to the vecintrin.h header file. gcc/ChangeLog: 2016-02-17 Andreas Krebbel * config/s390/vecintrin.h (vec_sub_u128): Define missing macro. --- gcc/config/s390/vecintrin.h | 1 + 1 file changed, 1 insertion(+) diff --git a/gcc/config/s390/vecintrin.h b/gcc/co

[PATCH 4/9] S/390: Adjust movstr-1.c testcase to work with the z13 stpcpy implementation.

2016-02-17 Andreas Krebbel * gcc.target/s390/md/movstr-1.c: Allow also the z13 strings instruction pattern name to prevent the testcase from failing with -march=z13. --- gcc/testsuite/gcc.target/s390/md/movstr-1.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) d

[PATCH 2/9] S/390: z13 lcbb fix address operand.

gcc/ChangeLog: 2016-02-17 Andreas Krebbel * config/s390/s390.md: Add missing output modifier for operand 1 to print it as address properly. --- gcc/config/s390/s390.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/gcc/config/s390/s390.md b/gcc/config/

[PATCH] Fix Cilk+ #pragma cilk grainsize preprocessing (PR c++/69826)

Hi! The following testcase works unless -save-temps or ccache is used (or manually performing -E and compilation separately). The problem is that #pragma cilk grainsize is supposed to have macro expansion (except for the grainsize keyword), but we weren't enabling that for -E. Fixed thusly, boot

[PATCH] Fix a C++ -Wnonnull-compare regression (PR c++/69850)

Hi! As -Wnonnull -> -Wnonnull-compare warning has been moved from FE to after going into SSA form, we have a problem because delete ptr; in C++ may introduce a comparison on its own, and on that artificial comparison can complain about comparison of nonnull argument with NULL. The following patch

Re: [6 Regession] Usage of unitialized pointer io/list_read.c (

On 02/16/2016 05:37 PM, Jerry DeLisle wrote: > See patch to fix this below. > Committed on trunk, r233500 after regression testing, -fsanitize=address testing, and valgrind testing. Jerry

[PATCH] Fix up avx512* regressions caused by the cse.c one-liner change (PR target/69671)

Hi! As I wrote in the PR, fwprop is able to forward CONST0_RTX back into instructions even if CSE optimized them, but the problem in that case is that for vector_move_operand "0C" operands if they appear inside of (vec_select ... (parallel [(const_int 0) ... ])) the result is also simplified, so o

Re: Partial Offloading (was: [hsa merge 07/10] IPA-HSA pass)

On Thu, Jan 28, 2016 at 12:36:19 +0100, Thomas Schwinge wrote: > I made an attempt to capture the recent discussion (plus my own > ideas/understanding) in this new section: > . Please > change/extend, as required. Thanks for summarizing this

Re: [RFC] [P2] [PR tree-optimization/33562] Lowering more complex assignments.

On 02/17/2016 07:13 AM, Richard Biener wrote: - /* Continue walking until we reach a kill. */ - while (!stmt_kills_ref_p (temp, ref)); + /* Continue walking until we reach a full kill as a single statement + or there are no more live bytes. */ + while (!stmt_kills_ref_p (temp, ref) +

Re: [PATCH] Fix driver handling of multiple -ftree-parallelize-loops= options (PR driver/69805)

On 02/17/2016 12:14 AM, Tom de Vries wrote: Here's the documentation entry for the gt spec function (I forgot to add it when introducing the function), using the new semantics. Copy-pasting from the resulting .info viewed in emacs for a human-readable version: ... 'gt' The 'gt'

[PATCH][AArch64][v2] Skip gcc.target/aarch64/assembler_arch_1.c if assembler does not support it

Hi all, I've thought about this check a bit more and I think we can compactly auto-generate checks for any aarch64 architecture extension support in the assembler. This is done in a similar way we autogenerate the arm_arch_*_ok checks for arm. So in this revision we autogenerate aarch64_asm__ok

Re: [PATCH, PR69607] Mark offload symbols as global in lto

On 17/02/16 13:30, Jakub Jelinek wrote: On Wed, Feb 17, 2016 at 01:02:17PM +0100, Tom de Vries wrote: Mark offload symbols as global in lto I'm really not familiar with that part of LTO, so I'm CCing Honza and Richard here. 2016-02-08 Tom de Vries PR lto/69607 * lto-partit

Re: [patch] fix docs for C++ warn_unused type attribute

On 02/17/2016 09:49 AM, Jonathan Wakely wrote: https://gcc.gnu.org/onlinedocs/gcc/C_002b_002b-Attributes.html says that attribute((warn_unused)) is not useful for std::mutex because it controls a resource. That's incorrect, std::mutex *is* a resource, but it doesn't control one. In fact declaring

Re: [PATCH][CilkPlus] Fix PR69363

On Wed, Feb 17, 2016 at 04:14:22PM +0100, Jakub Jelinek wrote: > On Wed, Feb 17, 2016 at 04:11:44PM +0100, Marek Polacek wrote: > > On Wed, Feb 17, 2016 at 06:08:14PM +0300, Ilya Verbin wrote: > > > > This line is too long. But you could have just done > > > > > > My editor shows exactly 80 chars

Re: [PATCH][CilkPlus] Fix PR69363

On 2016.02.17 at 18:29 +0300, Ilya Verbin wrote: > On Wed, Feb 17, 2016 at 16:28:34 +0100, Marek Polacek wrote: > > On Wed, Feb 17, 2016 at 04:14:22PM +0100, Jakub Jelinek wrote: > > > On Wed, Feb 17, 2016 at 04:11:44PM +0100, Marek Polacek wrote: > > > > On Wed, Feb 17, 2016 at 06:08:14PM +0300, I

Re: [PATCH] Add debug_function_to_file

On 17/02/16 14:42, Richard Biener wrote: On Wed, Feb 17, 2016 at 1:41 PM, Tom de Vries wrote: >Hi, > >once in a while I'm in a gdb debug session debugging cc1, and want to print >the current function to file. > >There's a debug function debug_function that prints a function to stderr, >and ther

Re: [PATCH] Add debug_function_graph_to_file

Richard> What does it take to write it in python instead? IIUC you're asking for it to display the function graph at a given point in time. This is easy - you can just dump it to a file and then run the appropriate visualization tool. Once upon a time I also wrote Python code to display a functi

Re: [PATCH][CilkPlus] Fix PR69363

On Wed, Feb 17, 2016 at 16:28:34 +0100, Marek Polacek wrote: > On Wed, Feb 17, 2016 at 04:14:22PM +0100, Jakub Jelinek wrote: > > On Wed, Feb 17, 2016 at 04:11:44PM +0100, Marek Polacek wrote: > > > On Wed, Feb 17, 2016 at 06:08:14PM +0300, Ilya Verbin wrote: > > > > > This line is too long. But y

Re: [PATCH][CilkPlus] Fix PR69363

On Wed, Feb 17, 2016 at 04:11:44PM +0100, Marek Polacek wrote: > On Wed, Feb 17, 2016 at 06:08:14PM +0300, Ilya Verbin wrote: > > > This line is too long. But you could have just done > > > > My editor shows exactly 80 chars. > > The maximum is 79. Well, check_GNU_style.sh complains just about

Re: [PATCH][CilkPlus] Fix PR69363

On Wed, Feb 17, 2016 at 06:08:14PM +0300, Ilya Verbin wrote: > > This line is too long. But you could have just done > > My editor shows exactly 80 chars. The maximum is 79. Marek

Re: [PATCH][CilkPlus] Fix PR69363

On Wed, Feb 17, 2016 at 15:46:00 +0100, Jakub Jelinek wrote: > On Wed, Feb 17, 2016 at 05:32:58PM +0300, Ilya Verbin wrote: > > + && !SCALAR_FLOAT_TYPE_P (TREE_TYPE (t)) > > + && TREE_CODE (TREE_TYPE (t)) != POINTER_TYPE) > > + { > > + error_at (OMP_CLA

Re: [wwwdocs] Describe behavior of -flifetime-dse in class constructors

On 02/17/2016 03:23 PM, Jakub Jelinek wrote: > "has been" looks weird. I'd say that the C++ compiler is now more > aggressive... > > Jakub Sending v3. M. Index: htdocs/gcc-6/porting_to.html === RCS file: /cvs/gcc/wwwdocs/htdo

[PATCH] Fix PR69568

So after pondering a while the only thing we can do now is force -fno-short-enums. Installed. Richard. 2016-02-17 Richard Biener PR testsuite/69586 * gcc.dg/uninit-21.c: Add -fno-short-enums. Index: gcc/testsuite/gcc.dg/uninit-21.c ==

[patch] fix docs for C++ warn_unused type attribute

https://gcc.gnu.org/onlinedocs/gcc/C_002b_002b-Attributes.html says that attribute((warn_unused)) is not useful for std::mutex because it controls a resource. That's incorrect, std::mutex *is* a resource, but it doesn't control one. In fact declaring std::mutex as a local variable and then never u

Re: [PATCH][CilkPlus] Fix PR69363

On Wed, Feb 17, 2016 at 05:32:58PM +0300, Ilya Verbin wrote: > This patch fixes > Bootstrap and make check passed. OK for... stage 1? Ok for stage1, with a few nits. > --- a/gcc/c/c-typeck.c > +++ b/gcc/c/c-typeck.c > @@ -12527,7 +12527,8 @@ c

[PATCH] Fix PR69854

The following patch fixes PR69854 - match.pd shouldn't use fold_unary/binary and expect no or a constant result. Instead using the now available const_binop/unop is recommended. Bootstrap and regtest running on x86_64-unknown-linux-gnu. Richard. 2016-02-17 Richard Biener PR middle

[PATCH][CilkPlus] Fix PR69363

Hi! This patch fixes Bootstrap and make check passed. OK for... stage 1? gcc/c-family/ PR c++/69363 * c-cilkplus.c (c_finish_cilk_clauses): Remove function. * c-common.h (c_finish_cilk_clauses): Remove declaration. gcc

RFA: Prevent an ICE when redeclaring a static function as weak

Hi Guys, Redefining a previously defined static function as both public and weak triggers an ICE in ipa-visibility.c: internal compiler error: in function_and_variable_visibility, at ipa-visibility.c:518 This bug has been discussed and patch proposed here: https://gcc.gnu.org/bugzilla/sh

Re: [wwwdocs] Describe behavior of -flifetime-dse in class constructors

On Wed, Feb 17, 2016 at 03:21:07PM +0100, Martin Liška wrote: > --- htdocs/gcc-6/porting_to.html 14 Feb 2016 13:13:43 - 1.14 > +++ htdocs/gcc-6/porting_to.html 17 Feb 2016 14:20:13 - > @@ -324,6 +324,52 @@ > -fabi-version or -Wabi option to disable or warn about. > > > +

Re: [wwwdocs] Describe behavior of -flifetime-dse in class constructors

On 02/16/2016 05:55 PM, Martin Sebor wrote: > I think the new text deserves a new heading of its own rather than > being added under the existing "Stricter flexible array member rules." > (The "Finally..." part changed by the patch still applies to the > flexible array members.) > > Martin Hi Mar

Re: [RFC] [P2] [PR tree-optimization/33562] Lowering more complex assignments.

On Wed, Feb 17, 2016 at 3:02 PM, Jeff Law wrote: > On 02/17/2016 03:48 AM, Richard Biener wrote: > >>> I instrumented a bootstrap -- the improved DSE finds ~20k additional DSE >>> opportunities during a GCC bootstrap that could not be found by the >>> current >>> DSE. Yes, 20k additional statemen

Re: [RFC] [P2] [PR tree-optimization/33562] Lowering more complex assignments.

On 02/17/2016 03:48 AM, Richard Biener wrote: I instrumented a bootstrap -- the improved DSE finds ~20k additional DSE opportunities during a GCC bootstrap that could not be found by the current DSE. Yes, 20k additional statements deleted by tree DSE. Yow! Well, DCE also can do quite some DS

Re: [PATCH] Add debug_function_graph_to_file

On Wed, Feb 17, 2016 at 2:51 PM, Marek Polacek wrote: > On Wed, Feb 17, 2016 at 02:45:36PM +0100, Richard Biener wrote: >> OTOH I have in my local trees a more convenient form (attached). >> >> (gdb) call debug_dot_cfg (cfun, 1<<6) >> >> and a X window with the dotted graph opens. > > Is there any

Re: [PATCH] Add a new option "-fmerge-bitfields" (patch / doc inside)

On Wed, 17 Feb 2016, Richard Biener wrote: > On Wed, 17 Feb 2016, Bernd Schmidt wrote: > > > > > > > On 02/17/2016 02:18 PM, Daniel Gutson wrote: > > > On Wed, Nov 26, 2014 at 5:46 AM, Andrew Pinski wrote: > > > > > > FYI. This causes gfc_add_interface_mapping in fortrant/trans-expr.c to > >

Re: [PATCH] Add debug_function_graph_to_file

On Wed, Feb 17, 2016 at 02:45:36PM +0100, Richard Biener wrote: > OTOH I have in my local trees a more convenient form (attached). > > (gdb) call debug_dot_cfg (cfun, 1<<6) > > and a X window with the dotted graph opens. Is there any chance we could get this into the mainline? I'd love to use t

Re: [PATCH] Add a new option "-fmerge-bitfields" (patch / doc inside)

On Wed, 17 Feb 2016, Bernd Schmidt wrote: > > > On 02/17/2016 02:18 PM, Daniel Gutson wrote: > > On Wed, Nov 26, 2014 at 5:46 AM, Andrew Pinski wrote: > > > > FYI. This causes gfc_add_interface_mapping in fortrant/trans-expr.c to > > > be miscompiled for aarch64-linux-gnu. I am still debuggin

Re: [PATCH] Add a new option "-fmerge-bitfields" (patch / doc inside)

On Wed, Feb 17, 2016 at 02:45:16PM +0100, Bernd Schmidt wrote: > > > On 02/17/2016 02:18 PM, Daniel Gutson wrote: > >On Wed, Nov 26, 2014 at 5:46 AM, Andrew Pinski wrote: > > >>FYI. This causes gfc_add_interface_mapping in fortrant/trans-expr.c to > >>be miscompiled for aarch64-linux-gnu. I am

Re: [PATCH] Add a new option "-fmerge-bitfields" (patch / doc inside)

On 02/17/2016 02:18 PM, Daniel Gutson wrote: On Wed, Nov 26, 2014 at 5:46 AM, Andrew Pinski wrote: FYI. This causes gfc_add_interface_mapping in fortrant/trans-expr.c to be miscompiled for aarch64-linux-gnu. I am still debugging it and trying to get a smaller testcase. Hello, is the

Re: [PATCH] Add debug_function_graph_to_file

On Wed, Feb 17, 2016 at 2:04 PM, Tom de Vries wrote: > Hi, > > once in a while I'm in a gdb debug session debugging cc1, and want to print > the current function graph to file (see also > https://gcc.gnu.org/ml/gcc-patches/2016-02/msg01160.html for the non-graph > variant). > > That is currently p

Re: [PATCH] Add debug_function_to_file

On Wed, Feb 17, 2016 at 1:41 PM, Tom de Vries wrote: > Hi, > > once in a while I'm in a gdb debug session debugging cc1, and want to print > the current function to file. > > There's a debug function debug_function that prints a function to stderr, > and there are methods to redirect output of a c

[wwwdocs] Fix broken links on /readings.html

The links to http://cm.bell-labs.com/cm/cs/who/dmr/ no logner work, these appear to be the right ones now. Committed to CVS. Index: htdocs/readings.html === RCS file: /cvs/gcc/wwwdocs/htdocs/readings.html,v retrieving revision 1.242

Re: [PATCH][ARM] PR target/69161: Don't ignore mode when matching comparison operator in cstore-like patterns

Hi Nick, On 17/02/16 13:13, Nick Clifton wrote: Hi Kyrill, Ok for trunk? 2016-01-29 Kyrylo Tkachov PR target/69161 * config/arm/predicates.md (arm_comparison_operator_mode): New predicate. * config/arm/arm.md (*mov_scc): Use arm_comparison_operator_mode instead of

Re: [PATCH] Add a new option "-fmerge-bitfields" (patch / doc inside)

On Wed, Nov 26, 2014 at 5:46 AM, Andrew Pinski wrote: > On Wed, Oct 29, 2014 at 6:34 AM, Zoran Jovanovic > wrote: >> Hello, >> This is new patch version in which reported issue is fixed. >> Also, patch is rebased to the revision 216452 and some minor code clean-up >> is done. > > FYI. This cause

Re: [Patch, fortran, pr67451, v1] [5/6 Regression] ICE with sourced allocation from coarray

Dear Andre, I had left this to somebody else, since I am travelling! The patch is verging on 'obvious' and so it is OK for trunk. Could you check the line terminators please? I am seeing CR-LFs but this might be an effect of transmission. Thanks for the patch. Paul On 10 February 2016 at 12:2

Re: [PATCH][ARM] PR target/69161: Don't ignore mode when matching comparison operator in cstore-like patterns

Hi Kyrill, > Ok for trunk? > > 2016-01-29 Kyrylo Tkachov > > PR target/69161 > * config/arm/predicates.md (arm_comparison_operator_mode): > New predicate. > * config/arm/arm.md (*mov_scc): Use arm_comparison_operator_mode > instead of arm_comparison_operator. > (*mov_n

[PATCH] Add debug_function_graph_to_file

Hi, once in a while I'm in a gdb debug session debugging cc1, and want to print the current function graph to file (see also https://gcc.gnu.org/ml/gcc-patches/2016-02/msg01160.html for the non-graph variant). That is currently possible by doing: ... (gdb) call clean_graph_dump_file ("foo.1"

[PATCH, ARM] Fix redefinition of cpp macros with #pragma GCC pop,reset

target_option_current_node, used in c-pragma.c to check if a state should be popped or reseted to the previous value, was not set when switching state with #pragma GCC target (I missed to see that, since it is done for pop,reset). So in some cases the state might not be reset correctly. This patch

[PATCH] Add debug_function_to_file

Hi, once in a while I'm in a gdb debug session debugging cc1, and want to print the current function to file. There's a debug function debug_function that prints a function to stderr, and there are methods to redirect output of a command to a file ( https://sourceware.org/gdb/onlinedocs/gdb/

Re: RFC: [Patch, PR Bug 60818] - ICE in validate_condition_mode on powerpc*-linux-gnu* ]

Hi Rohit, [ Please CC: me on combine patches ]. On Tue, Feb 16, 2016 at 05:02:30AM +, Rohit Arul Raj D wrote: > This is related to the following bug: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60818 > > Test case: > unsigned int ou; > int jv(void) > { > unsigned int rg; > return rg <

Re: [PATCH, PR69607] Mark offload symbols as global in lto

On Wed, Feb 17, 2016 at 01:02:17PM +0100, Tom de Vries wrote: > Mark offload symbols as global in lto I'm really not familiar with that part of LTO, so I'm CCing Honza and Richard here. > 2016-02-08 Tom de Vries > > PR lto/69607 > * lto-partition.c (promote_offload_tables): New fu

[PATCH] Fix PR69609, BB reorder slowness

This fixes BB reorder speed for the testcase (lots of computed gotos in a very large function). BB reorder time goes down from reorder blocks : 260.53 (80%) usr 0.37 (27%) sys 261.80 (80%) wall 432597 kB (58%) ggc to reorder blocks : 1.03 ( 2%) usr 0.08 ( 7%) sys

Re: [PATCH, PR69607] Mark offload symbols as global in lto

On 08/02/16 14:00, Tom de Vries wrote: Hi, when running libgomp.c testsuite with "-flto -flto-partition=1to1 -fno-toplevel-reorder" we run into many compilation failures like this: ... /tmp/.ltrans0.ltrans.o:(.gnu.offload_funcs+0x1a0): undefined reference to `MAIN__._omp_fn.0'^M ... The

[COMMITTED][AArch64] Add missing return in aarch64_internal_mov_immediate

As part of the final cleanup of aarch64_internal_mov_immediate a return was accidentally removed. This causes the 2-instruction case to fallthrough into the general case even when it found a match. An example immediate is 0xcccd which was using 2 instructions in GCC5 but now requires

[testsuite] Skip gcc.dg/ifcvt-4.c on Visium

Same reason as for ARM. Tested on visium-elf, applied on the mainline. 2016-02-17 Eric Botcazou * gcc.dg/ifcvt-4.c: Skip on Visium. -- Eric BotcazouIndex: gcc.dg/ifcvt-4.c === --- gcc.dg/ifcvt-4.c (revision 233448) +++

[visium] Tidy up support of machine-specific libfuncs

This adds an internal table instead of regenerating naked symbols each time the libfuncs are used. Tested on visium-elf, applied on the mainline. 2016-02-17 Eric Botcazou * config/visium/visium.c (machine_libfunc_index): New enum. (machine_libfuncs): New structure. (

Re: [RFC] [P2] [PR tree-optimization/33562] Lowering more complex assignments.

On Wed, Feb 17, 2016 at 8:30 AM, Jeff Law wrote: > On 02/14/2016 11:38 AM, Richard Biener wrote: >> >> On February 14, 2016 5:35:13 PM GMT+01:00, Jeff Law >> wrote: >>> >>> On 02/12/2016 10:21 AM, Jeff Law wrote: > > But really we simply need a better DSE algorithm. So the way t

Re: [PING] genattrab.c generate switch

On 19/01/16 10:44, Richard Biener wrote: On Mon, Jan 18, 2016 at 7:48 PM, Jeff Law wrote: On 01/18/2016 07:09 AM, Jesper Broge Jørgensen wrote: Ping patch: https://gcc.gnu.org/ml/gcc-patches/2016-01/msg00784.html I'd put it in my gcc-7 queue. But if Richard, Bernd, Richi or someone else wa

Re: [PATCH, ARM] stop changing signedness in PROMOTE_MODE

On 17/02/16 10:20, Christophe Lyon wrote: On 17 February 2016 at 11:05, Kyrill Tkachov wrote: On 17/02/16 10:03, Christophe Lyon wrote: On 15 February 2016 at 12:32, Kyrill Tkachov wrote: On 04/02/16 08:58, Ramana Radhakrishnan wrote: On Tue, Jun 30, 2015 at 2:15 AM, Jim Wilson wrote: Th

Re: [PATCH, ARM] stop changing signedness in PROMOTE_MODE

On 17 February 2016 at 11:05, Kyrill Tkachov wrote: > > On 17/02/16 10:03, Christophe Lyon wrote: >> >> On 15 February 2016 at 12:32, Kyrill Tkachov >> wrote: >>> >>> On 04/02/16 08:58, Ramana Radhakrishnan wrote: On Tue, Jun 30, 2015 at 2:15 AM, Jim Wilson wrote: > > This

Re: [PATCH][ARM][RFC] PR target/65578 Fix gcc.dg/torture/stackalign/builtin-apply-4.c for single-precision fpus

Ping. https://gcc.gnu.org/ml/gcc-patches/2016-02/msg00634.html As mentioned before, this is actually a fix for PR target/69538. I got confused when writing the cover letter and ChangeLog... Thanks, Kyrill On 09/02/16 17:24, Kyrill Tkachov wrote: On 09/02/16 17:21, Kyrill Tkachov wrote: Hi al

Re: [PATCH, ARM] stop changing signedness in PROMOTE_MODE

On 17/02/16 10:03, Christophe Lyon wrote: On 15 February 2016 at 12:32, Kyrill Tkachov wrote: On 04/02/16 08:58, Ramana Radhakrishnan wrote: On Tue, Jun 30, 2015 at 2:15 AM, Jim Wilson wrote: This is my suggested fix for PR 65932, which is a linux kernel miscompile with gcc-5.1. The proble

Re: [PATCH, ARM] stop changing signedness in PROMOTE_MODE

On 15 February 2016 at 12:32, Kyrill Tkachov wrote: > > On 04/02/16 08:58, Ramana Radhakrishnan wrote: >> >> On Tue, Jun 30, 2015 at 2:15 AM, Jim Wilson wrote: >>> >>> This is my suggested fix for PR 65932, which is a linux kernel >>> miscompile with gcc-5.1. >>> >>> The problem here is caused by

Commit: MSP430: Update devices list

Hi Guys, I am checking in this patch to update the list of MSP430 devices built in to the msp430 backend. Cheers Nick gcc/ChangeLog 2016-02-17 Nick Clifton * config/msp430/msp430.c (msp430_mcu_data): Sync with data from TI's devices.csv file as of March 2016. Index: gc

[Ada] Fix missing synchronization for Atomic_Components on array object

This plugs a small hole in the implementation of atomic synchronization: the compiler fails to enforce it if a pragma/aspect Has_Atomic_Components is put directly on an array object instead of an array type. It's not a regression but the issue is annoying and the fix is trivial. Tested on x86_64

Re: [PING, Patch, fortran, pr67451, v1] [5/6 Regression] ICE with sourced allocation from coarray

PING ! On Wed, 10 Feb 2016 12:26:24 +0100 Andre Vehreschild wrote: > Hi all, > > unfortunately was my last patch for pr67451 not perfect and introduced > regressions occurring on s390(x) and with the sanitizer. These were > caused, because when taking the array specs from the source=-expression

[PATCH] Fix PR69553

Honza added skipping for the (broken) type-comparing code in operand_equal_p for OEP_ADDRESS_OF. This exposes the fact that both IMAGPART_EXPR and ARRAY_REF lack comparison of their offsetting effect. In fact the issue is somewhat latent as for example nothing verified before that we don't compa

[Ada] Fix ICE on element of array with enumeration index

This is a regression present on the mainline. The compiler aborts on the element of an array whose index is an enumeration type with representation clause and whose component contains a small discriminated type with default discriminant, because of a spurious SAVE_EXPR created by gigi. Tested

Re: [6 Regession] Usage of unitialized pointer io/list_read.c (

On 16 February 2016 at 18:49, Jerry DeLisle wrote: > On 02/16/2016 12:06 AM, Christophe Lyon wrote: >> On 15 February 2016 at 23:16, Janne Blomqvist >> wrote: >>> On Mon, Feb 15, 2016 at 11:45 PM, Jerry DeLisle >>> wrote: The title of the PR should be "Mishandling of namelist comments" or