[Bug tree-optimization/68275] New: bb-slp-38 FAILs on armeb

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68275 Bug ID: 68275 Summary: bb-slp-38 FAILs on armeb Product: gcc Version: unknown Status: UNCONFIRMED Severity: normal Priority: P3 Component: tree-optimization

[Bug fortran/65454] Extending both forms of relational operators

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65454 --- Comment #4 from Paul Martin --- For information : The Silverfrost FTN95 compiler , version 7.20 compiles with no errors the program submitted in Comment 0 (though I had to delete two '::' separators). It gives the same results as the

Re: [PATCH 01/02] PR/62314: add ability to add fixit-hints

On 11/10/2015 05:35 PM, David Malcolm wrote: + /* Nasty workaround to convince the linker to add + rich_location::add_fixit_insert + rich_location::add_fixit_remove + rich_location::add_fixit_replace + to cc1 for use by diagnostic_plugin_test_show_locus, + before anything

Re: [Patch AArch64] Switch constant pools to separate rodata sections.

On Tue, Nov 10, 2015 at 4:39 PM, Alan Lawrence wrote: > On 04/11/15 14:26, Ramana Radhakrishnan wrote: >> >> >> True and I've just been reading more of the backend - We could now start >> using blocks for constant pools as well. So let's do that. >> >> How does something

[Bug c++/68265] Arbitrary syntactic nonsense silently accepted after 'int (*){}' until the next close brace

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68265 --- Comment #2 from Zack Weinberg --- This problem apparently goes back at least as far as 4.8. Stack Overflow people found a number of variations, please consult https://stackoverflow.com/questions/23033043/is-it-a-new-c11-style-of-comments

[Bug c/68272] Unwanted out-of-line instances for C inline functions that are also GCC builtins.

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68272 --- Comment #1 from joseph at codesourcery dot com --- This is not a standards conformance bug, on multiple grounds: * The C standard does not permit you to define your own copies of standard library functions (that is, functions in the

[Bug tree-optimization/68275] bb-slp-38 FAILs on armeb

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68275 --- Comment #2 from Christophe Lyon --- Created attachment 36679 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=36679=edit slp2 log, big-endian

[Bug tree-optimization/68275] bb-slp-38 FAILs on armeb

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68275 --- Comment #3 from Christophe Lyon --- Created attachment 36680 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=36680=edit slp2 log, little-endian

[Bug c++/54601] AIX uses atexit which causes unloading of shared modules to break

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=54601 --- Comment #14 from Hugo Koblmueller --- David, which version does/will include these recent additions? I recently encountered a crash on program exit in AIX 6.1, in a setup where I used a static C++ objects inside functions within a shared

[PATCH 01/02] PR/62314: add ability to add fixit-hints

This patch adds the ability to add "fix-it hints" to a rich_location, which will be displayed when the corresponding diagnostic is printed. It does not actually add any fix-it hints (that comes in the second patch), but it adds test coverage of the machinery and printing, by using the existing

[PATCH 02/02] C FE: add fix-it hint for . vs ->

This is the most trivial example of a real fix-it example I could think of: if the user writes ptr.field rather than ptr->field. gcc/c/ChangeLog: * c-typeck.c (build_component_ref): Special-case POINTER_TYPE when generating a "not a structure of union" error message, and

Re: [RFC][PATCH] Preferred rename register in regrename pass

On 10 November 2015 at 12:41, Robert Suchanek wrote: > Hi Christophe, > >> Hi, >> >> Since you committed this (r230087 if I'm correct), I can see that GCC >> fails to build >> ligfortran for target arm-none-linuxgnueabi --with-cpu=cortex-a9. > ... >> >> Can you have a

RE: [PATCH 1/2][ARC] Add support for ARCv2 CPUs

> If you can name a pre-existing testcase to trigger the assert, the patch is > approved for separate check-in. The patch solves the gcc.dg/pr29921-2.c error, visible for ARC700 architecture. I will prepare a new patch for this error. Thank you for the review, Claudiu

Re: [hsa 7/12] Disabling the vectorizer for GPU kernels/functions

On Tue, 10 Nov 2015, Martin Jambor wrote: > On Fri, Nov 06, 2015 at 09:38:21AM +0100, Richard Biener wrote: > > On Thu, 5 Nov 2015, Martin Jambor wrote: > > > > > Hi, > > > > > > in the previous email I wrote we need to "change behavior" of a few > > > optimization passes. One was the

Re: [AArch64] Move iterators from atomics.md to iterators.md

On Mon, Nov 02, 2015 at 11:44:02AM +, Matthew Wahab wrote: > Hello > > One of the review comments for the v8.1 atomics patches was that the > iterators and unspec declarations should be moved out of the atomics.md > file (https://gcc.gnu.org/ml/gcc-patches/2015-09/msg01375.html). > > The

[Bug bootstrap/68271] [6 Regression] Boostrap fails on x86_64-apple-darwin14 at r230084

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68271 --- Comment #11 from Jakub Jelinek --- (In reply to Richard Henderson from comment #10) > I believe the tokens didn't stay around in C at the time. > But I might be wrong... it was 9 years ago... > > If we can remove it, it does seem like a

[Bug c++/68195] gcc//ld produces invalid ABI results (cxx11 problem?)

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68195 Jonathan Wakely changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug target/68277] New: [5] [SH]: error: insn does not satisfy its constraints when compiling erlang

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68277 Bug ID: 68277 Summary: [5] [SH]: error: insn does not satisfy its constraints when compiling erlang Product: gcc Version: 5.2.1 URL:

Re: [PATCH 01/02] PR/62314: add ability to add fixit-hints

On Tue, 2015-11-10 at 17:26 +0100, Bernd Schmidt wrote: > On 11/10/2015 05:35 PM, David Malcolm wrote: > > + /* Nasty workaround to convince the linker to add > > + rich_location::add_fixit_insert > > + rich_location::add_fixit_remove > > + rich_location::add_fixit_replace > > +

[Bug libstdc++/68197] negative index to ios_base::iword lead to unpredictable result

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68197 --- Comment #1 from Jonathan Wakely --- I would argue that your program has undefined behaviour, there is no array element at a negative index.

[Bug bootstrap/68271] [6 Regression] Boostrap fails on x86_64-apple-darwin14 at r230084

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68271 --- Comment #10 from Richard Henderson --- I believe the tokens didn't stay around in C at the time. But I might be wrong... it was 9 years ago... If we can remove it, it does seem like a good idea.

Re: [PATCH] Simple optimization for MASK_STORE.

2015-11-10 17:46 GMT+03:00 Richard Biener : > On Tue, Nov 10, 2015 at 1:48 PM, Ilya Enkovich wrote: >> 2015-11-10 15:33 GMT+03:00 Richard Biener : >>> On Fri, Nov 6, 2015 at 2:28 PM, Yuri Rumyantsev

Re: [PR64164] drop copyrename, integrate into expand

On 05/11/15 05:08, Alexandre Oliva wrote: [PR67753] fix copy of PARALLEL entry_parm to CONCAT target_reg for gcc/ChangeLog PR rtl-optimization/67753 PR rtl-optimization/64164 * function.c (assign_parm_setup_block): Avoid allocating a stack slot if we don't have

[Bug tree-optimization/68274] __builtin_unreachable pessimizes code

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68274 --- Comment #2 from Matt Godbolt --- Thanks for updating the bug. As a corollary, moving the unreachability above the returns yields the same code as the non-unreachable: https://goo.gl/MdULOs -- int test_with_unreach_First(Side side, const Foo

Re: [Patch AArch64] Switch constant pools to separate rodata sections.

On 10/11/15 16:39, Alan Lawrence wrote: Since r229878, I've been seeing FAIL: gcc.dg/attr-weakref-1.c (test for excess errors) UNRESOLVED: gcc.dg/attr-weakref-1.c compilation failed to produce executable (both previously passing) on aarch64-none-elf, aarch64_be-none-elf, and

Re: [RFC] Combine vectorized loops with its scalar remainder.

On Tue, Nov 10, 2015 at 2:02 PM, Ilya Enkovich wrote: > 2015-11-10 15:30 GMT+03:00 Richard Biener : >> On Tue, Nov 3, 2015 at 1:08 PM, Yuri Rumyantsev wrote: >>> Richard, >>> >>> It looks like misunderstanding - we assume

[Bug c++/68266] pointers to arrays of excessive size not diagnosed

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68266 Marek Polacek changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|

[Bug tree-optimization/68274] __builtin_unreachable pessimizes code

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68274 Richard Biener changed: What|Removed |Added Keywords||missed-optimization

[Bug tree-optimization/68275] bb-slp-38 FAILs on armeb

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68275 --- Comment #1 from Christophe Lyon --- Created attachment 36678 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=36678=edit slp1 log, little-endian

[PATCH] libcpp: add examples to source_location description

This is a followup to: [PATCH 10/10] Compress short ranges into source_location which adds some worked examples of what a source_location/location_t can encode. Successfully bootstrapped on x86_64-pc-linux-gnu (although it only touches a comment). OK for trunk? libcpp/ChangeLog: *

[C/C++ PATCH] Reject declarators with huge arrays (PR c/68107, c++/68266)

While both C and C++ FEs are able to reject e.g. int a[__SIZE_MAX__ / sizeof(int)]; they are accepting code such as int (*a)[__SIZE_MAX__ / sizeof(int)]; As Joseph pointed out, any construction of a non-VLA type whose size is half or more of the address space should receive a compile-time error.

[Bug rtl-optimization/68185] [6 Regression] wrong code at -O3 on x86_64-linux-gnu (in 64-bit mode)

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68185 --- Comment #2 from Thomas Preud'homme --- Here's a quick update. What I found so far is that after split2, we have: (insn 148 61 304 16 (set (reg:CCNO 17 flags) (compare:CCNO (reg:HI 44 r15 [orig:91 pretmp_9 ] [91])

[Bug c++/54601] AIX uses atexit which causes unloading of shared modules to break

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=54601 --- Comment #13 from David Edelsohn --- The recent additions to GCC cxa atexit support on AIX may fix this.

[Bug ipa/68273] [5/6 Regression] Wrong code on mips/mipsel with -fipa-sra

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68273 Richard Biener changed: What|Removed |Added Keywords||wrong-code CC|

[Bug libstdc++/68190] [5/6 Regression] iterator mix up with set::find and heterogenous lookup

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68190 --- Comment #11 from Jonathan Wakely --- Author: redi Date: Tue Nov 10 15:12:24 2015 New Revision: 230113 URL: https://gcc.gnu.org/viewcvs?rev=230113=gcc=rev Log: Fix return type of heterogeneous find for sets PR libstdc++/68190

[patch] libstdc++/68190 Fix return type of heterogeneous find for sets

This converts the return type of heterogeneous find members to the correct set iterator type. Tested powerpc64le-linux, committed to trunk. Will commit to the gcc-5-branch too. commit d84e13dd8a7d47016bdfc5a9f45d8658a9d16ed9 Author: Jonathan Wakely Date: Tue Nov 10

[Bug bootstrap/68271] [6 Regression] Boostrap fails on x86_64-apple-darwin14 at r230084

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68271 Jason Merrill changed: What|Removed |Added CC||rth at gcc dot gnu.org --- Comment #9

[Bug libstdc++/68276] New: ios_base::_M_grow_words should use new (std::nothrow)

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68276 Bug ID: 68276 Summary: ios_base::_M_grow_words should use new (std::nothrow) Product: gcc Version: 5.2.1 Status: UNCONFIRMED Severity: normal Priority: P3

[Bug c++/54601] AIX uses atexit which causes unloading of shared modules to break

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=54601 --- Comment #15 from David Edelsohn --- GCC development trunk and it will be in GCC 5.3.

Re: [Patch AArch64] Switch constant pools to separate rodata sections.

On 04/11/15 14:26, Ramana Radhakrishnan wrote: True and I've just been reading more of the backend - We could now start using blocks for constant pools as well. So let's do that. How does something like this look ? Tested on aarch64-none-elf - no regressions. 2015-11-04 Ramana

Re: [PATCH] Simple optimization for MASK_STORE.

On Nov 10, 2015, at 6:56 AM, Ilya Enkovich wrote: > 2015-11-10 17:46 GMT+03:00 Richard Biener : >> On Tue, Nov 10, 2015 at 1:48 PM, Ilya Enkovich >> wrote: >>> 2015-11-10 15:33 GMT+03:00 Richard Biener

Re: [PATCH] Make BB vectorizer work on sub-BBs

On 10 November 2015 at 14:02, Richard Biener wrote: > On Tue, 10 Nov 2015, Christophe Lyon wrote: > >> On 6 November 2015 at 12:11, Kyrill Tkachov wrote: >> > Hi Richard, >> > >> > >> > On 06/11/15 11:09, Richard Biener wrote: >> >> >> >> On Fri, 6 Nov

Re: [gomp4] Random omp-low.c backporting

Hi Nathan! On Tue, 10 Nov 2015 09:19:50 -0500, Nathan Sidwell wrote: > I've committed this to backport a bunch of random bits from trunk to gomp4, > and > thereby reduce divergence. Yeah, I had some of these on my list, too. > --- omp-low.c (revision 230080) > +++ omp-low.c

Re: [hsa 7/12] Disabling the vectorizer for GPU kernels/functions

On Fri, Nov 06, 2015 at 09:38:21AM +0100, Richard Biener wrote: > On Thu, 5 Nov 2015, Martin Jambor wrote: > > > Hi, > > > > in the previous email I wrote we need to "change behavior" of a few > > optimization passes. One was the flattening of GPU functions and the > > other two are in the

[Bug bootstrap/68271] [6 Regression] Boostrap fails on x86_64-apple-darwin14 at r230084

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68271 --- Comment #8 from cesar at gcc dot gnu.org --- I'm not sure it will make much of a difference, but Thomas is planning on adding two openacc clauses bind and nohost. Is there anything I can do to help here, or is this already being taken care

Re: [PATCH 1/2] Fix invalid left shift of negative value.

On Nov 10, 2015, at 3:13 AM, Dominik Vogt wrote: > On Tue, Nov 10, 2015 at 12:11:23PM +0100, Dominik Vogt wrote: >> The following series of patches fixes all occurences of >> left-shifting negative constants in C code which is undefined by >> the C standard. The patches

Re: [PATCH 3a/4][AArch64] Add attribute for compatibility with ARM pipeline models

On Tue, Nov 10, 2015 at 6:01 PM, Ramana Radhakrishnan wrote: > On Tue, Nov 10, 2015 at 5:50 PM, Evandro Menezes > wrote: >>2015-11-10 Evandro Menezes >> >>gcc/ >> >>* config/aarch64/aarch64.md

Re: [PATCH], Add power9 support to GCC, patch #6 (IEEE 128-bit hardware support)

On Tue, Nov 10, 2015 at 12:41:07AM +, Joseph Myers wrote: > I don't see any conversions between KFmode and TImode (in either > direction, signed or unsigned) here - I suppose there are no instructions > for that? No in power9 there is no instruction that converts 128-bit integer to IEEE

[Bug go/68255] cgo-generated constructor not being called

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68255 Ian Lance Taylor changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|

Re: [PATCH 1/6] Use IFN_SQRT in tree-vect-patterns.c

On Tue, 10 Nov 2015, Richard Biener wrote: > Looks ok but I wonder if this is dead code with > > (for pows (POW) > sqrts (SQRT) > cbrts (CBRT) > (simplify > (pows @0 REAL_CST@1) > (with { > const REAL_VALUE_TYPE *value = TREE_REAL_CST_PTR (@1); > REAL_VALUE_TYPE tmp; >

[PATCH 3a/4][AArch64] Add attribute for compatibility with ARM pipeline models

2015-11-10 Evandro Menezes gcc/ * config/aarch64/aarch64.md (predicated): Copy attribute from "arm.md". This patch duplicates an attribute from arm.md so that the same pipeline model can be used for both AArch32 and AArch64. Bootstrapped on

[Bug libstdc++/68190] [5/6 Regression] iterator mix up with set::find and heterogenous lookup

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68190 --- Comment #12 from Jonathan Wakely --- Author: redi Date: Tue Nov 10 18:08:50 2015 New Revision: 230118 URL: https://gcc.gnu.org/viewcvs?rev=230118=gcc=rev Log: Fix return type of heterogeneous find for sets PR libstdc++/68190

Re: [PATCH, i386]: Fix gcc.c-torture/compile/pr41634.c FAIL

On Tue, Nov 10, 2015 at 7:00 PM, Richard Henderson wrote: > On 11/10/2015 06:54 PM, Uros Bizjak wrote: >> >> - return "movabs{}\t{%1, %0|%0, %1}"; >> + return "movabs{}\t{%1, %P0|[%P0], %1}"; > > > The thing that's missing from this, that's present in the patch that I

[Bug libstdc++/56158] bad enum values computed by operator~ in ios_base.h

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56158 --- Comment #13 from Jonathan Wakely --- N.B. we could also get rid of the _S_ios_xxx_end enumerators, but that would break any code which (foolishly) refers to them, e.g. to suppress Clang's -Wswitch warnings. My suggestion assumes that

Re: State of support for the ISO C++ Transactional Memory TS and remanining work

On 09/11/15 00:19, Torvald Riegel wrote: Hi, I'd like to summarize the current state of support for the TM TS, and outline the current plan for the work that remains to complete the support. I'm aware we're at the end of stage 1, but I'm confident we can still finish this work and hope to

[PATCH][ARM] PR 68149 Fix ICE in unaligned_loaddi split

Hi all, This ICE in this PR occurs when we're trying to split unaligned_loaddi into two SImode unaligned loads. The problem is in the addressing mode. When reload was picking the addressing mode we accepted an offset of -256 because the mode in the pattern is advertised as DImode and that was

[Bug libstdc++/68190] [5/6 Regression] iterator mix up with set::find and heterogenous lookup

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68190 Jonathan Wakely changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[PATCH, i386]: Fix gcc.c-torture/compile/pr41634.c FAIL

Hello! Recent AS patches introduced the above compilation failure. We have to treat movabs operands in a special way - without %rip and inside square bracket for -masm=intel. Also, the patch removes dead code while at it. 2015-11-10 Uros Bizjak * config/i386/i386.c

Re: [C/C++ PATCH] Reject declarators with huge arrays (PR c/68107, c++/68266)

On Tue, 10 Nov 2015, Marek Polacek wrote: > While both C and C++ FEs are able to reject e.g. > int a[__SIZE_MAX__ / sizeof(int)]; > they are accepting code such as > int (*a)[__SIZE_MAX__ / sizeof(int)]; > > As Joseph pointed out, any construction of a non-VLA type whose size is half > or >

Re: [PATCH 3a/4][AArch64] Add attribute for compatibility with ARM pipeline models

On Tue, Nov 10, 2015 at 5:50 PM, Evandro Menezes wrote: >2015-11-10 Evandro Menezes > >gcc/ > >* config/aarch64/aarch64.md (predicated): Copy attribute from >"arm.md". > > This patch duplicates an attribute from arm.md so

Re: [PATCH], Add power9 support to GCC, patch #8 (add integer multiply/add)

This patch adds support for the MADDLD instruciton, which is a fused multiply/add instruction for integers. At this time, it is for 64-bit multiplies only. Eventually, we will restructure 128-bit multiply so that we can use the 64x64 + 64 high bit varients. I have bootstrapped a compiler with

[PATCH 3b/4][AArch64] Add scheduling model for Exynos M1

2015-11-10 Evandro Menezes gcc/ * config/aarch64/aarch64-cores.def: Use the Exynos M1 sched model. * config/aarch64/aarch64.md: Include "exynos-m1.md". * config/arm/arm-cores.def: Use the Exynos M1 sched model. * config/arm/arm.md:

[PATCH] gcc.c: new macro POST_LINK_SPECS to be able to add additional steps after linking

One may need to execute extra steps after linking program. This is required for example for DJGPP to run stubify.exe on file generated by linker. The only way how to achieve was to use LINK_COMMAND_SPEC. It would be much easier and less error prone to use new macro POST_LINK_SPEC introduced in

Re: State of support for the ISO C++ Transactional Memory TS and remanining work

On Tue, 2015-11-10 at 17:26 +, Szabolcs Nagy wrote: > On 09/11/15 00:19, Torvald Riegel wrote: > > Hi, > > > > I'd like to summarize the current state of support for the TM TS, and > > outline the current plan for the work that remains to complete the > > support. > > > > I'm aware we're at

Re: [C/C++ PATCH] Reject declarators with huge arrays (PR c/68107, c++/68266)

Hi, On 11/10/2015 05:36 PM, Marek Polacek wrote: + + /* Did array size calculations overflow or does the array + cover more than half of the address-space? */ + if (COMPLETE_TYPE_P (type) + && TREE_CODE (TYPE_SIZE_UNIT (type)) ==

Re: [PATCH 1/2] simplify-rtx: Simplify trunc of and of shiftrt

On Tue, Nov 10, 2015 at 12:16:09PM +0100, Bernd Schmidt wrote: > On 11/09/2015 08:33 AM, Segher Boessenkool wrote: > >If we have > > > > (truncate:M1 (and:M2 (lshiftrt:M2 (x:M2) C) C2)) > > > >we can write it instead as > > > > (and:M1 (lshiftrt:M1 (truncate:M1 (x:M2)) C) C2) > > > > > >+

Re: [RFC][PATCH] Preferred rename register in regrename pass

On Tue, Nov 10, 2015 at 05:22:40PM +0100, Christophe Lyon wrote: > On 10 November 2015 at 12:41, Robert Suchanek > wrote: > > Hi Christophe, > > > >> Hi, > >> > >> Since you committed this (r230087 if I'm correct), I can see that GCC > >> fails to build > >> ligfortran

Re: [PATCH, i386]: Fix gcc.c-torture/compile/pr41634.c FAIL

On 11/10/2015 06:54 PM, Uros Bizjak wrote: - return "movabs{}\t{%1, %0|%0, %1}"; + return "movabs{}\t{%1, %P0|[%P0], %1}"; The thing that's missing from this, that's present in the patch that I sent you off-list, is the thing for Intel syntax. Would you prefer to just add that

Re: Enable pointer TBAA for LTO

> > Index: tree.c > > === > > --- tree.c (revision 229968) > > +++ tree.c (working copy) > > @@ -13198,6 +13198,7 @@ gimple_canonical_types_compatible_p (con > >/* If the types have been previously registered and found equal > >

Re: [PATCH, i386]: Fix gcc.c-torture/compile/pr41634.c FAIL

On Tue, Nov 10, 2015 at 7:14 PM, Uros Bizjak wrote: > On Tue, Nov 10, 2015 at 7:00 PM, Richard Henderson wrote: >> On 11/10/2015 06:54 PM, Uros Bizjak wrote: >>> >>> - return "movabs{}\t{%1, %0|%0, %1}"; >>> + return "movabs{}\t{%1, %P0|[%P0], %1}";

Re: [PATCH][ARM][cleanup] Remove uses of CONST_DOUBLE_HIGH/LOW

Hi Ramana, On 10/11/15 14:33, Ramana Radhakrishnan wrote: On Thu, Nov 5, 2015 at 9:32 AM, Kyrill Tkachov wrote: Hi all, This cleanup patch removes handling of CONST_DOUBLE rtxes that carry large integers. These should never be passed down from the midend and the arm

Re: [PATCH 02/02] C FE: add fix-it hint for . vs ->

On Tue, 10 Nov 2015, David Malcolm wrote: > This is the most trivial example of a real fix-it example I could think > of: if the user writes > ptr.field > rather than ptr->field. > > gcc/c/ChangeLog: > * c-typeck.c (build_component_ref): Special-case POINTER_TYPE when >

[Bug libstdc++/56158] bad enum values computed by operator~ in ios_base.h

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56158 --- Comment #12 from Jonathan Wakely --- Richard's patch changes the values returned by operator~ which is not desirable. To fix the underlying type to int in C++03 (so that all values of int will be valid values of the enumeration type) we can

Re: [PATCH 1/6] Use IFN_SQRT in tree-vect-patterns.c

On November 10, 2015 6:29:36 PM GMT+01:00, Joseph Myers wrote: >On Tue, 10 Nov 2015, Richard Biener wrote: > >> Looks ok but I wonder if this is dead code with >> >> (for pows (POW) >> sqrts (SQRT) >> cbrts (CBRT) >> (simplify >> (pows @0 REAL_CST@1) >>

Re: Replace match.pd DEFINE_MATH_FNs with auto-generated lists

On November 10, 2015 9:13:25 PM GMT+01:00, Richard Sandiford wrote: >Richard Biener writes: >> On Sat, Nov 7, 2015 at 2:23 PM, Richard Sandiford >> wrote: >>> diff --git a/gcc/genmatch.c b/gcc/genmatch.c >>>

libgo patch committed: always use --whole-archive in go tool

This patch changes the Go tool to always use --whole-archive when linking gccgo packages. This fixes cases where a Go package uses cgo to call C code in which the only referenced symbol is a C global variable. This is a backport of https://golang.org/cl/16775 in the master Go sources. This

[Bug go/68255] cgo-generated constructor not being called

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68255 --- Comment #3 from ian at gcc dot gnu.org --- Author: ian Date: Tue Nov 10 20:31:11 2015 New Revision: 230120 URL: https://gcc.gnu.org/viewcvs?rev=230120=gcc=rev Log: PR go/68255 cmd/go: always use --whole-archive for gccgo

Re: [patch 4/3] Header file reduction - Tools for contrib - second cut

Andrew, can you go ahead and commit those changes into contrib? I think in a subdirectory would be best so that you can include the README. Make sure the permissions are set correctly. Applying them as a patch kept mucking them up. header-tools or somesuch should be a fine directory name

Re: Replace match.pd DEFINE_MATH_FNs with auto-generated lists

Richard Biener writes: > On Sat, Nov 7, 2015 at 2:23 PM, Richard Sandiford > wrote: >> diff --git a/gcc/genmatch.c b/gcc/genmatch.c >> index cff32b0..7139476 100644 >> --- a/gcc/genmatch.c >> +++ b/gcc/genmatch.c >> @@ -4638,6 +4638,11 @@

[Bug middle-end/68279] ICE: in create_pw_aff_from_tree, at graphite-sese-to-poly.c:836

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68279 Joost VandeVondele changed: What|Removed |Added CC||Joost.VandeVondele at mat dot ethz

[Bug fortran/67496] trans-array.c sanitizer runtime error: load of value 124, which is not a valid value for type 'bool'

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67496 Dominique d'Humieres changed: What|Removed |Added Status|UNCONFIRMED |WAITING Last reconfirmed|

[v3 PATCH] LWG 2510, make the default constructors of library tag types explicit.

Tested on Linux-X64. 2015-11-10 Ville Voutilainen LWG 2510, make the default constructors of library tag types explicit. * include/bits/mutex.h (defer_lock_t, try_lock_t, adopt_lock_t): Add an explicit default constructor. *

Re: [PATCH, applied], Add power9 support to GCC, patch #9 (config.gcc)

I applied this patch as obvious. I missed submitting it in my original patch for the power9 support (it was in the sandbox I was testing power9 support on). 2015-11-10 Michael Meissner * config.gcc (powerpc*-*-*, rs6000*-*-*): Add power9 to hosts that

Re: [RFC] [PATCH V2]: RE: [RFC] [Patch] Relax tree-if-conv.c trap assumptions.

On November 10, 2015 1:02:57 PM GMT+01:00, Richard Biener wrote: >On Sat, Nov 7, 2015 at 12:41 PM, Kumar, Venkataramanan > wrote: >> Hi Richard, >> >> I have now implemented storing of DR and references using hash maps. >> Please find

Re: [C/C++ PATCH] Reject declarators with huge arrays (PR c/68107, c++/68266)

On 11/10/2015 09:36 AM, Marek Polacek wrote: While both C and C++ FEs are able to reject e.g. int a[__SIZE_MAX__ / sizeof(int)]; they are accepting code such as int (*a)[__SIZE_MAX__ / sizeof(int)]; As Joseph pointed out, any construction of a non-VLA type whose size is half or more of the

[PATCH] Fix minor fallout from output_address changes

The ft32 and moxie ports failed to build after the recent output_address changes. Fixed thusly and committed to the trunk after verifying the moxie and ft32 ports in config-list.mk build again. Jeff commit b408dd85568c5d0c0a9673810280a8438753b60f Author: law

Re: [PATCH] libcpp: add examples to source_location description

On 11/10/2015 09:44 AM, David Malcolm wrote: This is a followup to: [PATCH 10/10] Compress short ranges into source_location which adds some worked examples of what a source_location/location_t can encode. Successfully bootstrapped on x86_64-pc-linux-gnu (although it only touches a comment).

[Bug c++/68278] New: internal compiler error with C++14 polymorphic lambda and type alias

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68278 Bug ID: 68278 Summary: internal compiler error with C++14 polymorphic lambda and type alias Product: gcc Version: 5.2.0 Status: UNCONFIRMED Severity: normal

[gomp4] Fix some broken tests

I've committed this to gomp4. In preparing the reworked firstprivate patch changes for gomp4's gimplify.c I discovered these testcases were passing by accident, and lacked a data clause. nathan 2015-11-10 Nathan Sidwell *

[PATCH, rs6000] Remove redundant logic from rs6000_secondary_reload_direct_move

Hi, While investigating another issue, I observed some repeated logic in rs6000_secondary_reload_direct_move (). This patch takes it out. No functional change intended, and quite straightforward, so I'll plan to commit shortly if no concerns are raised. Bootstrapped and tested on

Re: [PATCH 1/2] simplify-rtx: Simplify trunc of and of shiftrt

On 11/10/2015 06:44 PM, Segher Boessenkool wrote: Yes I know. All the rest of the code around is it like this though. Do you want this written in a saner way? I won't object to leaving it as-is for now, but in the future it would be good to keep this in mind. I'm not entirely sure what

[Bug sanitizer/68065] Size calculations for VLAs can overflow

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68065 --- Comment #17 from Daniel Micay --- It's well-defined C code though. It shouldn't be possible to for anything to go wrong here when using -fstack-check, i.e. it should be guaranteed to trigger a stack overflow which is caught. The size

[Bug fortran/67498] interface.c sanitizer runtime error: load of value 1818451807, which is not a valid value for type 'expr_t'

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67498 Dominique d'Humieres changed: What|Removed |Added Status|UNCONFIRMED |WAITING Last reconfirmed|

Re: [PATCH][combine][RFC] Don't transform sign and zero extends inside mults

On Mon, Nov 09, 2015 at 03:51:32AM -0600, Segher Boessenkool wrote: > > >From the original patch submission, it looks that this patch would > > also benefit x86_32. > > Yes, that is what I thought too. > > > Regarding the above code size increase - do you perhaps have a > > testcase, to see

Re: [PATCH v4] SH FDPIC backend support

On Tue, Oct 27, 2015 at 11:01:39PM +0900, Oleg Endo wrote: > On Mon, 2015-10-26 at 22:47 -0400, Rich Felker wrote: > > On Sun, Oct 25, 2015 at 11:28:51PM +0900, Oleg Endo wrote: > > > On Fri, 2015-10-23 at 02:32 -0400, Rich Felker wrote: > > > > Here's my updated version of the FDPIC patch with

Re: [gomp4] Random omp-low.c backporting

On 11/10/15 11:28, Thomas Schwinge wrote: Hi Nathan! On Tue, 10 Nov 2015 09:19:50 -0500, Nathan Sidwell wrote: I've committed this to backport a bunch of random bits from trunk to gomp4, and thereby reduce divergence. Yeah, I had some of these on my list, too. ---

Re: [PATCH] gcc.c: new macro POST_LINK_SPECS to be able to add additional steps after linking

On 11/10/2015 11:16 AM, Andris Pavenis wrote: One may need to execute extra steps after linking program. This is required for example for DJGPP to run stubify.exe on file generated by linker. The only way how to achieve was to use LINK_COMMAND_SPEC. It would be much easier and less error prone

[Bug go/68255] cgo-generated constructor not being called

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68255 Ian Lance Taylor changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug middle-end/68279] New: ICE: in create_pw_aff_from_tree, at graphite-sese-to-poly.c:836

Priority: P3 Component: middle-end Assignee: unassigned at gcc dot gnu.org Reporter: Joost.VandeVondele at mat dot ethz.ch Target Milestone: --- with gcc version 6.0.0 20151110 (experimental) [trunk revision 230080] (GCC) > gfortran -c -O2 -floop-nest-optim

Re: Short-cut generation of simple built-in functions

Richard Biener writes: > On Sat, Nov 7, 2015 at 2:31 PM, Richard Sandiford > wrote: >> This patch short-circuits the builtins.c expansion code for a particular >> gimple call if: >> >> - the function has an associated internal function >> -

  1   2   3   4   >