[PATCH][PR rtl-optimization/78812] Fix pruning of expressions with embedded MEMs

2017-01-04 Thread Jeff Law
GCSE has some smarts to detect cases when a MEM appears in a block with predecessor edges marked as EDGE_ABNORMAL. Such MEMs are removed from the antic/transparent bitmaps. Unfortunately this code did not work if the MEM was not the topmost RTX. So something like (zero_extend (mem (...))

[Bug middle-end/78995] A strange copy error caused by O3 optimization

2017-01-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78995 Andrew Pinski changed: What|Removed |Added Status|UNCONFIRMED |WAITING Last reconfirmed|

[Bug rtl-optimization/78812] [5/6/7 Regression] Wrong code generation due to hoisting memory load across function call

2017-01-04 Thread law at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78812 --- Comment #14 from Jeffrey A. Law --- Author: law Date: Thu Jan 5 07:38:48 2017 New Revision: 244093 URL: https://gcc.gnu.org/viewcvs?rev=244093=gcc=rev Log: PR tree-optimizatin/78812 * rtl.h (contains_mem_rtx_p): Prototype.

[Bug debug/79000] New: ICE: in gen_member_die, at dwarf2out.c:23995

2017-01-04 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79000 Bug ID: 79000 Summary: ICE: in gen_member_die, at dwarf2out.c:23995 Product: gcc Version: 7.0 Status: UNCONFIRMED Keywords: lto Severity: normal Priority: P3

[Bug c++/78999] New: problem with gcc on cygwin??? cygwin 2.6.1 with gcc 5.4.0

2017-01-04 Thread bplummer at hotmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78999 Bug ID: 78999 Summary: problem with gcc on cygwin??? cygwin 2.6.1 with gcc 5.4.0 Product: gcc Version: 5.4.0 Status: UNCONFIRMED Severity: normal

Re: [PR tree-optimization/71691] Fix unswitching in presence of maybe-undef SSA_NAMEs (take 2)

2017-01-04 Thread Trevor Saunders
On Fri, Dec 16, 2016 at 09:41:33AM -0500, Aldy Hernandez wrote: > Hi folks. > > This is a follow-up on Jeff and Richi's interaction on the aforementioned PR > here: > > https://gcc.gnu.org/ml/gcc-patches/2016-08/msg01397.html > > I decided to explore the idea of analyzing may-undefness

Re: [bootstrap-O3,fortran] silence warning in simplify_transformation_to_array

2017-01-04 Thread Alexandre Oliva
On Jan 4, 2017, Alexandre Oliva wrote: > I'll prepare and post a patch anyway, but do we want to make it > standard practice? Here it is. simplify_transformation_to_array had the nested loop unrolled 7 times, which is reasonable given that it iterates over arrays of size

[bootstrap-O1] change value type to avoid sprintf buffer size warning (was: Re: [bootstrap-O1] enlarge sprintf output buffer to avoid warning)

2017-01-04 Thread Alexandre Oliva
On Jan 4, 2017, Martin Sebor wrote: > The manual recommends to use a length modifier to constrain the length > of output to that of a narrower type: > sprintf (xname, "", ((unsigned short)((uintptr_t)(t) & 0x))); > This should work even without optimization. It

Re: [-fcompare-debug] var tracking options are not optimization options

2017-01-04 Thread Alexandre Oliva
On Jan 4, 2017, Alexandre Oliva wrote: > So I guess we need some alternate PerFunction option flag that makes > it per-function, but not part of the ICF hash? Like this... If we include them in the ICF hash, they may cause congruence_groups to be processed in a different

[Bug middle-end/78914] missing -Wnonnull for a trivial null pointer dereference

2017-01-04 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78914 Martin Sebor changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug c/16351] NULL dereference warnings

2017-01-04 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=16351 Martin Sebor changed: What|Removed |Added CC||msebor at gcc dot gnu.org --- Comment

[Bug middle-end/78998] missing -Wnonnull for an unconditional call to strlen with a null argument

2017-01-04 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78998 Martin Sebor changed: What|Removed |Added Keywords||diagnostic See Also|

[Bug middle-end/78998] New: missing -Wnonnull for an unconditional call to strlen with a null argument

2017-01-04 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78998 Bug ID: 78998 Summary: missing -Wnonnull for an unconditional call to strlen with a null argument Product: gcc Version: 7.0 Status: UNCONFIRMED Severity:

[Bug libstdc++/78996] uses macro as name

2017-01-04 Thread timshen at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78996 Tim Shen changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED CC|

[Bug libstdc++/78996] uses macro as name

2017-01-04 Thread timshen at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78996 --- Comment #2 from Tim Shen --- Author: timshen Date: Thu Jan 5 03:18:17 2017 New Revision: 244092 URL: https://gcc.gnu.org/viewcvs?rev=244092=gcc=rev Log: 2017-01-05 Tim Shen PR libstdc++/78996 *

Re: [PATCH] s/__unused/__dimensions/ in

2017-01-04 Thread Tim Shen via gcc-patches
On Wed, Jan 4, 2017 at 7:16 PM, Tim Shen wrote: > Since it's an obvious textual fix, I'm going to check it in directly. Checked in with 80-columns limit respected. I have confusing tabstop settings... :/ -- Regards, Tim Shen

[PATCH] s/__unused/__dimensions/ in

2017-01-04 Thread Tim Shen via gcc-patches
Since it's an obvious textual fix, I'm going to check it in directly. -- Regards, Tim Shen commit b005916b043f52feed32b646f86ced80f3e5392b Author: Tim Shen Date: Wed Jan 4 19:07:07 2017 -0800 2017-01-05 Tim Shen PR

[Bug c/78987] Wrong location of a binary expression for -Waddress

2017-01-04 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78987 Martin Sebor changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug c++/64767] Could GCC warn when a pointer is compared against '\0'?

2017-01-04 Thread egall at gwmail dot gwu.edu
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64767 --- Comment #11 from Eric Gallager --- Thank you for adding this!

[Bug c++/78948] [C++17] constexpr if instantiating too eagerly

2017-01-04 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78948 Martin Sebor changed: What|Removed |Added Keywords||rejects-valid

[Bug c++/78988] Wrong location of a binary expression for -Waddress

2017-01-04 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78988 Martin Sebor changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug tree-optimization/78997] [7.0 regression] ICE on valid code at -O3 on x86_64-linux-gnu: verify_gimple failed

2017-01-04 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78997 Martin Sebor changed: What|Removed |Added Keywords||ice-on-valid-code

[Bug tree-optimization/78997] New: ICE on valid code at -O3 on x86_64-linux-gnu: verify_gimple failed

2017-01-04 Thread su at cs dot ucdavis.edu
: posix gcc version 7.0.0 20170104 (experimental) [trunk revision 244072] (GCC) $ $ gcc-trunk -O2 small.c $ gcc-6.2 -O3 small.c $ $ gcc-trunk -O3 small.c small.c: In function ‘foo’: small.c:7:6: error: the first argument of a VEC_COND_EXPR must be of a boolean vector type of the same number of elements

[Bug libstdc++/78996] uses macro as name

2017-01-04 Thread webrown.cpp at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78996 --- Comment #1 from W E Brown --- Created attachment 40465 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40465=edit Preprocessed source

[Bug libstdc++/78996] New: uses macro as name

2017-01-04 Thread webrown.cpp at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78996 Bug ID: 78996 Summary: uses macro as name Product: gcc Version: 7.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: libstdc++ Assignee:

[Bug c++/78995] New: A strange copy error caused by O3 optimization

2017-01-04 Thread zhangfei1 at cffex dot com.cn
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78995 Bug ID: 78995 Summary: A strange copy error caused by O3 optimization Product: gcc Version: 4.8.5 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

[Bug target/78823] Poor code on PowerPC when moving SFmode values between GPRs and vector registers

2017-01-04 Thread meissner at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78823 --- Comment #3 from Michael Meissner --- Author: meissner Date: Thu Jan 5 00:43:53 2017 New Revision: 244084 URL: https://gcc.gnu.org/viewcvs?rev=244084=gcc=rev Log: [gcc] 2017-01-04 Michael Meissner PR

[Bug target/70568] [5/6/7 regression] PowerPC64: union of floating and fixed doesn't use POWER8 GPR/VSR moves

2017-01-04 Thread meissner at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70568 --- Comment #8 from Michael Meissner --- Author: meissner Date: Thu Jan 5 00:43:53 2017 New Revision: 244084 URL: https://gcc.gnu.org/viewcvs?rev=244084=gcc=rev Log: [gcc] 2017-01-04 Michael Meissner PR

[Bug target/71977] powerpc64: Use VSR when operating on float and integer

2017-01-04 Thread meissner at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71977 --- Comment #4 from Michael Meissner --- Author: meissner Date: Thu Jan 5 00:43:53 2017 New Revision: 244084 URL: https://gcc.gnu.org/viewcvs?rev=244084=gcc=rev Log: [gcc] 2017-01-04 Michael Meissner PR

Re: [PATCH], PR 71977/70568/78823: Improve PowerPC code that uses SFmode in unions

2017-01-04 Thread David Edelsohn
On Wed, Jan 4, 2017 at 4:00 PM, Michael Meissner wrote: > On Wed, Jan 04, 2017 at 02:17:16PM -0500, David Edelsohn wrote: >> The change to rs6000_emit_move() really should have been in a helper >> function. We have to stop adding to the complexity of the function. I

[Bug other/32415] libgcc_s not found in library search path with --enable-version-specific-runtime-libs

2017-01-04 Thread bruno at clisp dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=32415 bruno at clisp dot org changed: What|Removed |Added CC||bruno at clisp dot org ---

[Bug middle-end/78993] False positive from -Wmaybe-uninitialized

2017-01-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78993 --- Comment #4 from Andrew Pinski --- # i_5 = PHI # j_27 = PHI # prephitmp_7 = PHI <0(3), prephitmp_17(4)> _14 = i_5 > 9; _18 = prephitmp_7 | _14; if (_18 != 0) goto ; [44.99%] else

[Bug middle-end/78993] False positive from -Wmaybe-uninitialized

2017-01-04 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78993 Martin Sebor changed: What|Removed |Added Known to fail||4.1.0, 5.3.0, 6.2.0, 7.0 --- Comment #3

[Bug middle-end/78993] False positive from -Wmaybe-uninitialized

2017-01-04 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78993 Martin Sebor changed: What|Removed |Added Keywords||diagnostic

[Bug rtl-optimization/78812] [5/6/7 Regression] Wrong code generation due to hoisting memory load across function call

2017-01-04 Thread law at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78812 --- Comment #13 from Jeffrey A. Law --- So I see a case in postreload-gcse.c where we might mis-handle when the destination is a ZERO_EXTRACT or STRICT_LOW_PART. Neither happen often which is probably why we've never noticed.

Re: MAINTAINERS

2017-01-04 Thread Gerald Pfeifer
On Tue, 11 Oct 2016, Nathan Sidwell wrote: Update my email address Did you mean to step down as -type-safe vectors Nathan Sidwell maintainer? Unlike the other entries you modified, there is no + line with your new address. Gerald

[Bug target/78994] -Ofast makes aarch64 C++ benchmark slower for A53

2017-01-04 Thread tulipawn at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78994 --- Comment #3 from PeteVine --- Hey, that works for me too! (62565 vs 70758 in favour of -Ofast). Usefully strange :)

[Bug rtl-optimization/78812] [5/6/7 Regression] Wrong code generation due to hoisting memory load across function call

2017-01-04 Thread law at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78812 --- Comment #12 from Jeffrey A. Law --- I'm mostly concerned about other places where we assume that a memory reference is supposed to show up at the toplevel of a source/dest. For example, it looks like we don't properly handle the case where

Re: [PATCH v2] Add mcpu flag for Qualcomm falkor core

2017-01-04 Thread Gerald Pfeifer
On Thu, 10 Nov 2016, Richard Earnshaw wrote: >> * gcc/config/aarch64/aarch64-cores.def (qdf24xx): Update part >> number. >> (falkor): New core. > Installed. Please can you prepare a patch for the release notes as well. In case you are wondering, Siddhesh, Richard was referring to

[Bug rtl-optimization/78812] [5/6/7 Regression] Wrong code generation due to hoisting memory load across function call

2017-01-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78812 --- Comment #11 from Jakub Jelinek --- (In reply to Jeffrey A. Law from comment #10) > Since that's not a MEM_P, the expression isn't removed from antic/transp > which makes it subject to hoisting across the abnormal edge. > > This could be

[Bug c/78989] Missing -Waddress warning

2017-01-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78989 Andrew Pinski changed: What|Removed |Added Depends on||77513 --- Comment #6 from Andrew Pinski

[Bug c/78989] Missing -Waddress warning

2017-01-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78989 --- Comment #5 from Andrew Pinski --- Isn't there a dup somewhere? I Know this was filed before.

[Bug rtl-optimization/78812] [5/6/7 Regression] Wrong code generation due to hoisting memory load across function call

2017-01-04 Thread law at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78812 Jeffrey A. Law changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |law at redhat dot com ---

[Bug c++/60685] exception not caught by enclosing catch

2017-01-04 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60685 Jonathan Wakely changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug target/78994] -Ofast makes aarch64 C++ benchmark slower for A53

2017-01-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78994 Andrew Pinski changed: What|Removed |Added Summary|-Ofast makes aarch64 C++|-Ofast makes aarch64 C++

[Bug target/78994] -Ofast makes aarch64 C++ benchmark slower

2017-01-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78994 Andrew Pinski changed: What|Removed |Added URL|https://github.com/supercur | |io/dsp-bench-cpp

[Bug middle-end/78994] New: -Ofast makes aarch64 C++ benchmark slower

2017-01-04 Thread tulipawn at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78994 Bug ID: 78994 Summary: -Ofast makes aarch64 C++ benchmark slower Product: gcc Version: 5.4.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

[Bug libstdc++/78991] std::sort and std::unique can not use std::function with clang++ -std=c++14

2017-01-04 Thread daiw at gmx dot net
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78991 Tobias changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

Re: [C++ PATCH] Reject invalid reference type non-static data members in unions (PR c++/78890)

2017-01-04 Thread Jakub Jelinek
On Wed, Jan 04, 2017 at 04:24:55PM -0500, Jason Merrill wrote: > On Mon, Jan 2, 2017 at 2:32 PM, Jakub Jelinek wrote: > > if (TREE_CODE (type) == REFERENCE_TYPE) > > { > > - error ("in C++98 %q+D may not have reference type %qT " > > -

Re: [C++ PATCH] Fix decomp handling of fields with reference type (PR c++/78931)

2017-01-04 Thread Jakub Jelinek
On Wed, Jan 04, 2017 at 04:27:42PM -0500, Jason Merrill wrote: > On Thu, Dec 29, 2016 at 10:11 AM, Jakub Jelinek wrote: > > probe = TREE_OPERAND (probe, 0); > > TREE_TYPE (v[i]) = TREE_TYPE (probe); > > layout_decl (v[i], 0); > > -

[Bug libstdc++/78991] std::sort and std::unique can not use std::function with clang++ -std=c++14

2017-01-04 Thread daiw at gmx dot net
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78991 --- Comment #2 from Tobias --- Thanks. The evidence you collected shows quite clear, that it probably is a problem with clang. So I now posted it here: https://llvm.org/bugs/show_bug.cgi?id=31537

[Bug c++/64767] Could GCC warn when a pointer is compared against '\0'?

2017-01-04 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64767 Marek Polacek changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug c++/64767] Could GCC warn when a pointer is compared against '\0'?

2017-01-04 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64767 --- Comment #9 from Marek Polacek --- Author: mpolacek Date: Wed Jan 4 21:47:04 2017 New Revision: 244076 URL: https://gcc.gnu.org/viewcvs?rev=244076=gcc=rev Log: PR c++/64767 * c.opt (Wpointer-compare): New option. *

[Bug tree-optimization/78910] Wrong print-return-value for a negative number

2017-01-04 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78910 Martin Sebor changed: What|Removed |Added Keywords||patch --- Comment #2 from Martin Sebor

[PATCH] correct off-by-one error in handling of precision of negative numbers (PR 78910)

2017-01-04 Thread Martin Sebor
The attached patch corrects an off-by-one mistake in handling non-zero precisions in signed conversions (%d and %i) with negative arguments, such as in sprintf(d, "%.2", -1). The call which results in the three bytes "-01" on output, not 2. I.e., the minus sign must be taken into consideration.

[Bug libstdc++/78991] std::sort and std::unique can not use std::function with clang++ -std=c++14

2017-01-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78991 --- Comment #1 from Andrew Pinski --- How positive you are that this is a libstdc++ bug rather than a clang bug? It works correctly with GCC 5.4.0's front-end and GCC 7.0's libstdc++ and front-end.

Re: [bootstrap-O3] add a default initializer to avoid a warning at -O3

2017-01-04 Thread David Malcolm
On Wed, 2017-01-04 at 09:04 -0700, Jeff Law wrote: > On 01/04/2017 06:42 AM, Jason Merrill wrote: > > On Wed, Jan 4, 2017 at 8:13 AM, Alexandre Oliva > > wrote: > > > On Jan 3, 2017, Jason Merrill wrote: > > > > > > > Are there bugzillas for these false

[Bug middle-end/78993] False positive from -Wmaybe-uninitialized

2017-01-04 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78993 --- Comment #1 from David Malcolm --- Created attachment 40462 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40462=edit Gimple dump from when warning is emitted

[Bug c++/78949] incorrect "unused variable" warning with SSE2

2017-01-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78949 --- Comment #7 from Jakub Jelinek --- Fixed on the trunk so far.

[Bug c++/78693] [6 Regression] Bogus 'inconsistent deduction for ‘auto’' error when having a dependent initializer and a nondependent one in the same declaration

2017-01-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78693 Jakub Jelinek changed: What|Removed |Added Summary|[6/7 Regression] Bogus |[6 Regression] Bogus

[Bug middle-end/78993] New: False positive from -Wmaybe-uninitialized

2017-01-04 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78993 Bug ID: 78993 Summary: False positive from -Wmaybe-uninitialized Product: gcc Version: 7.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

Re: [libcc1] add support for C++

2017-01-04 Thread Alexandre Oliva
On Dec 14, 2016, Alexandre Oliva wrote: > On Oct 19, 2016, Alexandre Oliva wrote: >> On Sep 23, 2016, Alexandre Oliva wrote: >>> This patchset adds support for the C++ language to libcc1. >>> It updates a few patches for libcc1 by Jan

[Bug c++/78949] incorrect "unused variable" warning with SSE2

2017-01-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78949 --- Comment #6 from Jakub Jelinek --- Author: jakub Date: Wed Jan 4 21:34:27 2017 New Revision: 244075 URL: https://gcc.gnu.org/viewcvs?rev=244075=gcc=rev Log: PR c++/78949 * typeck.c (cp_build_unary_op): Call mark_rvalue_use

[Bug c++/78693] [6/7 Regression] Bogus 'inconsistent deduction for ‘auto’' error when having a dependent initializer and a nondependent one in the same declaration

2017-01-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78693 --- Comment #5 from Jakub Jelinek --- Author: jakub Date: Wed Jan 4 21:30:35 2017 New Revision: 244074 URL: https://gcc.gnu.org/viewcvs?rev=244074=gcc=rev Log: PR c++/78693 * parser.c (cp_parser_simple_declaration): Only

Re: [C++ PATCH] Fix decomp handling of fields with reference type (PR c++/78931)

2017-01-04 Thread Jason Merrill
On Thu, Dec 29, 2016 at 10:11 AM, Jakub Jelinek wrote: > probe = TREE_OPERAND (probe, 0); > TREE_TYPE (v[i]) = TREE_TYPE (probe); > layout_decl (v[i], 0); > - SET_DECL_VALUE_EXPR (v[i], tt); > + SET_DECL_VALUE_EXPR (v[i],

Re: [C++ PATCH] Fix -Wunused-but-set-* false positive with ~ of vector type (PR c++/78949)

2017-01-04 Thread Jason Merrill
OK. On Thu, Dec 29, 2016 at 10:19 AM, Jakub Jelinek wrote: > Hi! > > For integral arg, mark_exp_read is called during > cp_perform_integral_promotions, for complex type it is called during > cp_default_conversion, but for vector types nothing actually calls it. > > Fixed

Re: [C++ PATCH] Reject invalid reference type non-static data members in unions (PR c++/78890)

2017-01-04 Thread Jason Merrill
On Mon, Jan 2, 2017 at 2:32 PM, Jakub Jelinek wrote: > if (TREE_CODE (type) == REFERENCE_TYPE) > { > - error ("in C++98 %q+D may not have reference type %qT " > -"because it is a member of a union", x, type); > -

Re: [C++ PATCH] Don't reject auto deduction of multiple variables if some initializers were type dependent (PR c++/78693)

2017-01-04 Thread Jason Merrill
OK. On Mon, Jan 2, 2017 at 3:11 PM, Jakub Jelinek wrote: > Hi! > > As the testcase shows, if some initializers are type dependent, auto_result > is still using auto, is not yet deduced and the deduction will happen during > instantiation. So rejecting it due to inconsistent

Re: [bootstrap-O3,fortran] silence warning in simplify_transformation_to_array

2017-01-04 Thread Alexandre Oliva
On Jan 4, 2017, Richard Biener wrote: >> Unfortunately I don't have another that silences the warning without >> (or even with) impact on codegen. > #pragma GCC diagnostic push ("Wno-array-bounds") > #pragma GCC diagnostic pop Wwwhaaat? #pragma GCC diagnostic to

Re: C/C++ PATCH to implement -Wpointer-compare warning (PR c++/64767) (version 2)

2017-01-04 Thread Joseph Myers
On Wed, 4 Jan 2017, Marek Polacek wrote: > Bootstrapped/regtested on x86_64-linux, ok for trunk? The C changes are OK. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH] Error on Enum option without RejectNegative

2017-01-04 Thread Joseph Myers
On Wed, 4 Jan 2017, Jakub Jelinek wrote: > On Wed, Jan 04, 2017 at 01:31:28PM +0100, Richard Biener wrote: > > > Rather than implicit RejectNegative it might be better to just diagnose > > > such options as invalid. If you agree, I can implement that as follow-up. > > > Also note that

Re: [PATCH] vimrc: fix TAB settings

2017-01-04 Thread Gerald Pfeifer
On Mon, 14 Nov 2016, Martin Liška wrote: > Following patch adds TAB settings to contrib/vimrc file. > Hope it looks reasonable? This does not appear applied, are you waiting for approval? If so, for something like this in contrib we don't need to get overly strict and I'd say go ahead.

[PATCH] Improve -f{,no}-vect-cost-model

2017-01-04 Thread Jakub Jelinek
On Wed, Jan 04, 2017 at 01:31:28PM +0100, Richard Biener wrote: > > And not sure why this actually > > is RejectNegative, wouldn't > > Common Alias(fvect-cost-model=,dynamic,unlimited) > > work just on fvect-cost-model (can test that)? > > Good question ;) If it works, ok. And here is a patch

[PATCH] Error on Enum option without RejectNegative

2017-01-04 Thread Jakub Jelinek
On Wed, Jan 04, 2017 at 01:31:28PM +0100, Richard Biener wrote: > > Rather than implicit RejectNegative it might be better to just diagnose > > such options as invalid. If you agree, I can implement that as follow-up. > > Also note that RejectNegative is only needed on the Enum switches that have

Re: [PATCH] PR sanitizer/78992: Fix sigaction definition on 32-bit sparc

2017-01-04 Thread Jakub Jelinek
On Wed, Jan 04, 2017 at 08:56:07PM +, James Clarke wrote: > libsanitizer: > PR sanitizer/78992 > * sanitizer_common/sanitizer_platform_limits_posix.h: sigaction > should only have __glibc_reserved0 as a member on 64-bit sparc. It should be committed upstream first and then

Re: C/C++ PATCH to implement -Wpointer-compare warning (PR c++/64767) (version 2)

2017-01-04 Thread Jason Merrill
On Wed, Jan 4, 2017 at 1:19 PM, Marek Polacek wrote: > +Note that the code above is illegal in C++11. "invalid". The C++ changes are OK. Jason

Re: [PATCH], PR 71977/70568/78823: Improve PowerPC code that uses SFmode in unions

2017-01-04 Thread Michael Meissner
On Wed, Jan 04, 2017 at 02:17:16PM -0500, David Edelsohn wrote: > The change to rs6000_emit_move() really should have been in a helper > function. We have to stop adding to the complexity of the function. I > won't insist that you split it out, but any addition of more than a > few lines in

Re: [PATCH] c++/78765

2017-01-04 Thread Jason Merrill
OK. Jason On Wed, Jan 4, 2017 at 11:44 AM, Nathan Sidwell wrote: > On 01/04/2017 10:13 AM, Jason Merrill wrote: > >> OK, that looks like the problem; we shouldn't be calling >> maybe_constant_value before we perform the conversion. > > > Yup, that worked. > > > ok? > > nathan >

[PATCH] PR sanitizer/78992: Fix sigaction definition on 32-bit sparc

2017-01-04 Thread James Clarke
libsanitizer: PR sanitizer/78992 * sanitizer_common/sanitizer_platform_limits_posix.h: sigaction should only have __glibc_reserved0 as a member on 64-bit sparc. --- libsanitizer/sanitizer_common/sanitizer_platform_limits_posix.h | 7 +-- 1 file changed, 5

[Bug sanitizer/78992] New: Incorrect sigaction definition on 32-bit sparc

2017-01-04 Thread jrtc27 at jrtc27 dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78992 Bug ID: 78992 Summary: Incorrect sigaction definition on 32-bit sparc Product: gcc Version: 7.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

[Bug rtl-optimization/78116] [7 regression] Performance drop after r241173 on avx512 target

2017-01-04 Thread pthaugen at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78116 --- Comment #10 from Pat Haugen --- (In reply to Jakub Jelinek from comment #9) > Any progress on this? Besides waiting for pr77536 to be fixed, I'm not sure what specifically can be done on this issue to fix the problem. I personally have not

[Bug libstdc++/78991] New: std::sort and std::unique can not use std::function with clang++ -std=c++14

2017-01-04 Thread daiw at gmx dot net
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78991 Bug ID: 78991 Summary: std::sort and std::unique can not use std::function with clang++ -std=c++14 Product: gcc Version: unknown Status: UNCONFIRMED Severity:

[Bug driver/78957] ICE: SIGSEGV with -fno-sso-struct=web

2017-01-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78957 Jakub Jelinek changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug driver/78957] ICE: SIGSEGV with -fno-sso-struct=web

2017-01-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78957 --- Comment #2 from Jakub Jelinek --- Author: jakub Date: Wed Jan 4 20:25:13 2017 New Revision: 244072 URL: https://gcc.gnu.org/viewcvs?rev=244072=gcc=rev Log: PR driver/78957 * c.opt (fsso-struct=): Add RejectNegative.

[Bug c++/71182] [6 Regression] parser.c cp_lexer_previous_token sanitizer detects member call on null pointer

2017-01-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71182 Jakub Jelinek changed: What|Removed |Added Summary|[6/7 Regression] parser.c |[6 Regression] parser.c

Re: [PATCH], PR 71977/70568/78823: Improve PowerPC code that uses SFmode in unions

2017-01-04 Thread Michael Meissner
On Wed, Jan 04, 2017 at 02:17:16PM -0500, David Edelsohn wrote: > On Thu, Dec 29, 2016 at 7:24 PM, Michael Meissner > wrote: > > Mike, > > Thanks for the tremendous effort on this patch. A few comments. > > The ChangeLog contains a lot of extraneous commentary

[Bug c++/71182] [6/7 Regression] parser.c cp_lexer_previous_token sanitizer detects member call on null pointer

2017-01-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71182 --- Comment #8 from Jakub Jelinek --- Author: jakub Date: Wed Jan 4 20:05:14 2017 New Revision: 244070 URL: https://gcc.gnu.org/viewcvs?rev=244070=gcc=rev Log: PR c++/71182 * parser.c (cp_lexer_previous_token): Use

[Bug target/78953] Errors in compiling Spec 2006 for power9

2017-01-04 Thread meissner at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78953 Michael Meissner changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug target/78900] ICE in gcc.target/powerpc/signbit-3.c

2017-01-04 Thread meissner at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78900 --- Comment #3 from Michael Meissner --- Fixed in trunk in subversion id 244044. I will hold the bug open until it is checked into the GCC 6 branch.

[Bug target/78056] [7 Regression] build failure on Power7

2017-01-04 Thread kelvin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78056 --- Comment #20 from kelvin at gcc dot gnu.org --- Author: kelvin Date: Wed Jan 4 20:03:00 2017 New Revision: 244068 URL: https://gcc.gnu.org/viewcvs?rev=244068=gcc=rev Log: gcc/testsuite/ChangeLog: 2017-01-04 Kelvin Nilsen

Re: [PATCH] C++: fix fix-it hints for misspellings within explicit namespaces

2017-01-04 Thread Jason Merrill
On Tue, Jan 3, 2017 at 8:28 PM, David Malcolm wrote: > PR c++/77829 and PR c++/78656 identify an issue within the C++ frontend > where it issues nonsensical fix-it hints for misspelled name lookups > within an explicitly given namespace: it finds the closest name within > all

[Bug c/78989] Missing -Waddress warning

2017-01-04 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78989 --- Comment #4 from David Malcolm --- ...or to use a rich location to send two locations for the warning, giving: return (asan_poison_variables && ^~ # 6 "gimplify.cpp" 3 4 __null

[Bug c/78989] Missing -Waddress warning

2017-01-04 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78989 --- Comment #3 from David Malcolm --- Looking at the PRs you filed about the locations (PR78987 and PR78988), perhaps the best approach here is for the location of the warning to be either this: return (asan_poison_variables &&

Re: [PATCH] Change DWARF5 .debug_loclists location description sizes from 2-byte length to uleb128 lengths

2017-01-04 Thread Jason Merrill
OK. On Tue, Jan 3, 2017 at 6:15 PM, Jakub Jelinek wrote: > Hi! > > http://dwarfstd.org/ShowIssue.php?issue=161102.1 > got accepted today, so DWARF5 is going to use uleb128 sizes instead of > 2-byte sizes in .debug_loclists section. > On a randomly chosen *.i file I had around,

Re: [C++ PATCH] Avoid UB in cp_lexer_previous_token (PR c++/71182)

2017-01-04 Thread Jason Merrill
OK. On Tue, Jan 3, 2017 at 5:42 PM, Jakub Jelinek wrote: > Hi! > > cp_lexer_new_from_tokens creates lexer that has NULL lexer->buffer, > calling lexer->buffer->address () therefore is UB (diagnosed by > --with-boot-config=bootstrap-ubsan). > > The following patch fixes this, or

Re: [C++ PATCH] Reject invalid auto foo (), a = 5;

2017-01-04 Thread Jason Merrill
On Tue, Jan 3, 2017 at 5:39 PM, Jakub Jelinek wrote: > + if (auto_function_declaration > + && (TREE_CODE (decl) == FUNCTION_DECL > + || auto_function_declaration != error_mark_node)) > + { > + error_at

Re: [patch,avr] PR78883: Implement a dummy scheduler

2017-01-04 Thread Jeff Law
On 01/04/2017 12:18 PM, Segher Boessenkool wrote: On Wed, Jan 04, 2017 at 06:42:23PM +, Richard Sandiford wrote: 1. reload has a bug that no-one really wants to fix (understandable) 2. the bug is triggered by paradoxical subregs of mems 3. those subregs are normally disabled on targets that

Re: [PATCH] Special case clzl like builtins in factor_out_conditional_conversion (PR tree-optimization/71016)

2017-01-04 Thread Jeff Law
On 01/04/2017 11:01 AM, Jakub Jelinek wrote: On Wed, Jan 04, 2017 at 11:19:46AM +0100, Richard Biener wrote: For the SSA_NAME + INTEGER_CST case restrict it to the case if (x_1 > 5) tem_2 = (char) x_1; # tem_3 = PHI that is, (char) x_1 uses x_1 and that also appears in the

Re: [PR tree-optimization/67955] Exploit PTA in DSE

2017-01-04 Thread Jeff Law
On 01/04/2017 11:55 AM, Jeff Law wrote: On 12/09/2016 01:28 AM, Richard Biener wrote: On Wed, Dec 7, 2016 at 12:18 AM, Jeff Law wrote: So I was going through the various DSE related bugs as stumbled across 67955. The basic issue in 67955 is that DSE is too simplistic when

[Bug tree-optimization/67955] tree-dse does not use pointer info

2017-01-04 Thread law at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67955 --- Comment #11 from Jeffrey A. Law --- Author: law Date: Wed Jan 4 19:22:44 2017 New Revision: 244067 URL: https://gcc.gnu.org/viewcvs?rev=244067=gcc=rev Log: PR tree-optimizatin/67955 * tree-ssa-alias.c

  1   2   3   >