[Bug target/88352] x86 mingw returning struct with just double in ST0 instead of edx:eax

2018-12-05 Thread jay.krell at cornell dot edu
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88352 --- Comment #2 from Jay --- The linked bug was amd64. This is x86. I'm not sure they are the same. Maybe.

[Bug sanitizer/80953] Support libsanitizer on Solaris

2018-12-05 Thread ebotcazou at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80953 --- Comment #22 from Eric Botcazou --- > I think it is important to find out why there are those differences in line > numbers. Is libbacktrace broken on Solaris, or not used at all, something > different? AFAICS they only occur with

[PATCH] Fix *ivdep* tests on SPARC etc. (PR testsuite/88369)

2018-12-05 Thread Jakub Jelinek
Hi! On (at least some of these) tests and on some targets, GCC prints messages like: ./cc1 -quiet -O3 -fopt-info-vec-optimized -mvis vect-ivdep-1.c vect-ivdep-1.c:11:3: optimized: loop vectorized using 8 byte vectors vect-ivdep-1.c:11:3: optimized: loop versioned for vectorization to enhance

[PATCH] Fix VRP with -fno-delete-null-pointer-checks (PR c/88367)

2018-12-05 Thread Jakub Jelinek
Hi! If we consider -fno-delete-null-pointer-checks as a way to support e.g. AVR and other targets which can validly place objects at NULL rather than a way to workaround UBs in code, I believe the following testcase must pass if there is e.g. char a[32]; // And this object ends up at

[PATCH] Don't optimize successive divs if there is also a mod with the same last arg (PR tree-optimization/85726)

2018-12-05 Thread Jakub Jelinek
Hi! This is my proposal for fixing this PR, just a heuristics when optimizing successive divides might not be a good idea (first testcase), and includes Marc's testcases which showed cases where optimizing successive divides is a good idea even if the inner divide is not a single use.

[PATCH] Handle clobber stmts in convert_nonlocal_reference_stmt (PR fortran/88304)

2018-12-05 Thread Jakub Jelinek
Hi! The following patch handles clobber stmts the same how we handle them in convert_local_reference_stmt, if it is the clobber with decl on the lhs and that lhs needs to be replaced by a field inside of a structure, the clobber is replaced by GIMPLE_NOP. Bootstrapped/regtested on x86_64-linux

[Bug middle-end/86121] [9 Regression] missing -Wstringop-overflow on strcpy followed by strcat

2018-12-05 Thread law at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86121 Jeffrey A. Law changed: What|Removed |Added Status|NEW |RESOLVED CC|

Re: add taishanv110 pipeline scheduling

2018-12-05 Thread Terry Guo
On Thu, Dec 6, 2018 at 9:31 AM wuyuan (E) wrote: > > Hi ARM maintainers: > The taishanv110 core uses generic pipeline scheduling, which > restricted the performance of taishanv110 core. By adding the pipeline > scheduling of taishanv110 core in GCC,The performance of taishanv110 has

[Bug target/86497] [8/9 regression] wasted instructions for x86 float x!=x

2018-12-05 Thread law at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86497 Jeffrey A. Law changed: What|Removed |Added Status|NEW |RESOLVED CC|

[Bug target/84251] [8/9 Regression] Performance regression in gcc 8/9 when comparing floating point numbers

2018-12-05 Thread law at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84251 Jeffrey A. Law changed: What|Removed |Added CC||no...@turm-lahnstein.de --- Comment #9

[Bug target/86973] ICE in expand_call, at calls.c:4217

2018-12-05 Thread law at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86973 Jeffrey A. Law changed: What|Removed |Added CC||law at redhat dot com

[Bug c++/87075] [7/8 Regression] ICE when compiling the test suite of GLM 0.9.9.0

2018-12-05 Thread law at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87075 Jeffrey A. Law changed: What|Removed |Added CC||law at redhat dot com

[Bug c/88383] ICE calling _builtin_has_attribute with an expression

2018-12-05 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88383 Martin Sebor changed: What|Removed |Added Keywords||patch Summary|ICE calling

[PATCH] handle expressions in __builtin_has_attribute (PR 88383)

2018-12-05 Thread Martin Sebor
The __builtin_has_attribute function fails with an ICE when its first argument is an expression such as INDIRECT_REF, or many others. The code simply assumes it's either a type or a decl. The attached patch corrects this oversight. While testing the fix I also noticed the C++ front end expects

[Bug c++/88384] __alignof__ of an rvalue is different between C and C++

2018-12-05 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88384 Martin Sebor changed: What|Removed |Added Keywords||wrong-code --- Comment #2 from Martin

Re: [PATCH 2/2] asm inline

2018-12-05 Thread Segher Boessenkool
Hi! On Tue, Dec 04, 2018 at 03:30:47PM +, Richard Sandiford wrote: > Segher Boessenkool writes: > Hmm, so we allow top-level "asm volatile" in C++ but not C? Apparently. Evert top-level asm is effectively volatile, so this doesn't really matter; but should we try to resolve the

[Bug c++/88385] New: [9 regression] ICE in tsubst_pack_expansion

2018-12-05 Thread s...@li-snyder.org
++ Assignee: unassigned at gcc dot gnu.org Reporter: s...@li-snyder.org Target Milestone: --- hi - gcc 20181205 gets an ICE compiling this code. (8.2.1 compiles it without error.) thanks, sss -- template struct xtest

Re: [PATCH, Fortran] pad char to int conversions with spaces instead of zeros (legacy)

2018-12-05 Thread Jerry DeLisle
On 12/4/18 9:04 AM, Fritz Reese wrote: On Tue, Dec 4, 2018 at 10:12 AM Jakub Jelinek wrote: Just a couple of random comments. -fdec-pad-with-spaces option name doesn't look right, because it doesn't say what the option affects. So perhaps have transfer in the option name? [...] Wouldn't it

[Bug c++/88384] __alignof__ of an rvalue is different between C and C++

2018-12-05 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88384 Martin Sebor changed: What|Removed |Added See Also||https://gcc.gnu.org/bugzill

[Bug c++/88384] New: __alignof__ of an rvalue is different between C and C++

2018-12-05 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88384 Bug ID: 88384 Summary: __alignof__ of an rvalue is different between C and C++ Product: gcc Version: 9.0 Status: UNCONFIRMED Severity: normal

add taishanv110 pipeline scheduling

2018-12-05 Thread wuyuan (E)
Hi ARM maintainers: The taishanv110 core uses generic pipeline scheduling, which restricted the performance of taishanv110 core. By adding the pipeline scheduling of taishanv110 core in GCC,The performance of taishanv110 has been improved. The patch as follows, please join.

Re: [PATCH] [PR87012] canonicalize ref type for tmpl arg

2018-12-05 Thread Jason Merrill
On 11/16/18 5:32 PM, Alexandre Oliva wrote: When binding an object to a template parameter of reference type, we take the address of the object and dereference that address. The type of the address may still carry (template) typedefs, but verify_unstripped_args_1 rejects such typedefs other

Re: [PATCH] [PR86747] tsubst friend tpl ctxt before looking it up for dupes

2018-12-05 Thread Jason Merrill
On 11/16/18 5:32 PM, Alexandre Oliva wrote: When a member template is redeclared as a friend, we enter the context of the member before looking it up, and then we check that the decls are compatible. However, when the member template references template types of the enclosing context, say an

Re: [PATCH[ Fix pr87269.C testcase

2018-12-05 Thread Jason Merrill
On 11/16/18 3:50 PM, Jakub Jelinek wrote: On Fri, Nov 16, 2018 at 10:01:05AM -0500, Nathan Sidwell wrote: 2018-11-16 Nathan Sidwell PR c++/87269 * parser.c (lookup_literal_operator): Mark overload for keeping when inside template. Refactor. *

Re: [PATCH] [PR86823] retain deferred access checks from outside firewall

2018-12-05 Thread Jason Merrill
On 11/16/18 9:16 PM, Alexandre Oliva wrote: We used to preserve deferred access check along with resolved template ids, but a tentative parsing firewall introduced additional layers of deferred access checks, so that we don't preserve the checks we want to any more. This patch collapses the

Re: [PATCH v2] C++: improvements to diagnostics using %P (more PR c++/85110)

2018-12-05 Thread Jason Merrill
On 12/3/18 5:54 PM, David Malcolm wrote: I was going to ping this patch: https://gcc.gnu.org/ml/gcc-patches/2018-11/msg00875.html but it has bit-rotted somewhat, so here's a refreshed version of it. Successfully bootstrapped & regrtested on x86_64-pc-linux-gnu. OK for trunk? Thanks Dave

[Bug rtl-optimization/85770] [9 Regression] ICE: in lra_eliminate, at lra-eliminations.c:1439 with -march=nano-1000

2018-12-05 Thread law at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85770 Jeffrey A. Law changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[committed] Add test for PR85770.

2018-12-05 Thread Jeff Law
PR85770 is fixed by Segher's combiner patch to avoid combining hard regs. Presumably it helps because it gives the allocators more freedom. I'm adding the testcase from the PR to the regression suite. Jeff commit 40fc691eac0ea9414f7908826c91afc70ff78617 Author: law Date: Thu Dec 6 00:40:08

[Bug rtl-optimization/85770] [9 Regression] ICE: in lra_eliminate, at lra-eliminations.c:1439 with -march=nano-1000

2018-12-05 Thread law at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85770 --- Comment #5 from Jeffrey A. Law --- Author: law Date: Thu Dec 6 00:40:08 2018 New Revision: 266839 URL: https://gcc.gnu.org/viewcvs?rev=266839=gcc=rev Log: PR rtl-optimization/85770 * gcc.target/i386/pr85770.c: New test.

Re: Fortran patches

2018-12-05 Thread Steve Kargl
On Wed, Dec 05, 2018 at 04:48:28PM -0500, Fritz Reese wrote: > On Wed, Dec 5, 2018 at 12:00 AM Steve Kargl > wrote: > > > > I intend to commit the attached patch on Saturday. > > Thanks for the work. I assume the patch bootstraps and passes > regression tests? The patch passed regression

Re: [PATCH]: Remove #ifdef PCC_BITFIELD_TYPE_MATTERS check from dwarf2out.c

2018-12-05 Thread Jeff Law
On 12/5/18 1:56 PM, Uros Bizjak wrote: > This macro is always defined through defaults.h. We can remove #ifdef > check from dwarf2out.c > > 2018-12-05 Uros Bizjak > > * dwarf2out.c (field_byte_offset): Remove > #ifdef PCC_BITFIELD_TYPE_MATTERS check. > > Bootstrapped and regression

Re: [patch,openacc] Propagate independent clause for OpenACC kernels pass

2018-12-05 Thread Julian Brown
On Tue, 4 Dec 2018 14:55:03 +0100 Richard Biener wrote: > On Tue, 4 Dec 2018, Jakub Jelinek wrote: > > > On Mon, Dec 03, 2018 at 11:40:39PM +, Julian Brown wrote: > > > Jakub asked in the following email at the time of the patch > > > submission for the gomp4 branch what the difference

[Bug rtl-optimization/85770] [9 Regression] ICE: in lra_eliminate, at lra-eliminations.c:1439 with -march=nano-1000

2018-12-05 Thread law at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85770 Jeffrey A. Law changed: What|Removed |Added CC||law at redhat dot com --- Comment #4

Re: Regarding projects and begineer guide.

2018-12-05 Thread Durgesh Kumar
Thanks, I'll check them out. On Thu 6 Dec, 2018, 1:51 AM Eric Gallager, wrote: > On 12/5/18, David Malcolm wrote: > > On Wed, 2018-12-05 at 22:48 +0530, Durgesh Kumar wrote: > >> Dear sir/mam > >> I am durgesh kumar from ABES engineering collenge,ghaziabad,uttar > >> pradesh,india. > >> > >> I

Question on Disable no throw for -flive-patching master option.

2018-12-05 Thread Qing Zhao
Hi, Honza, I have one question relate to whether to disable nothrow for -flive-patching as following: actually, there are two passes here: 1. local nothrow pass: in this pass, nothrow attribute is set locally after analyzing every stmt of the function body: unsigned int

Re: [PING #4][PATCH] avoid warning on constant strncpy until next statement is reachable (PR 87028)

2018-12-05 Thread Jeff Law
On 11/29/18 4:43 PM, Martin Sebor wrote: > On 11/29/18 4:07 PM, Jeff Law wrote: >> On 11/29/18 1:34 PM, Martin Sebor wrote: >>> On 11/16/2018 02:07 AM, Richard Biener wrote: On Fri, Nov 16, 2018 at 4:12 AM Martin Sebor  wrote: > > Ping: 

[Bug c/87028] false positive -Wstringop-truncation strncpy with global variable source string

2018-12-05 Thread law at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87028 Jeffrey A. Law changed: What|Removed |Added Status|ASSIGNED|RESOLVED CC|

libgo patch committed: Add precise stack scan support

2018-12-05 Thread Ian Lance Taylor
This libgo patch by Cherry Zhang adds support for precise stack scanning to the Go runtime. This uses per-function stack maps stored in the exception tables in the language-specific data area. The compiler needs to generate these stack maps; currently this is only done by a version of LLVM, not

[Bug c/87028] false positive -Wstringop-truncation strncpy with global variable source string

2018-12-05 Thread law at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87028 --- Comment #8 from Jeffrey A. Law --- Author: law Date: Wed Dec 5 23:10:08 2018 New Revision: 266833 URL: https://gcc.gnu.org/viewcvs?rev=266833=gcc=rev Log: PR c/87028 * calls.c (get_attr_nonstring_decl): Avoid setting *REF

Re: [PATCH 1/2] asm qualifiers (PR55681)

2018-12-05 Thread Segher Boessenkool
Hi! On Wed, Dec 05, 2018 at 04:47:37PM -0500, Jason Merrill wrote: > On 12/2/18 11:38 AM, Segher Boessenkool wrote: > >PR55681 observes that currently only one qualifier is allowed for > >inline asm, so that e.g. "volatile asm" is allowed, "const asm" is also > >okay (with a warning), but "const

[Bug c/88383] ICE calling _builtin_has_attribute(r, aligned(N))) on an overaligned reference r

2018-12-05 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88383 Martin Sebor changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|

Re: [C++ Patch] Fix grokbitfield location

2018-12-05 Thread Jason Merrill
On 12/5/18 5:34 PM, Paolo Carlini wrote: Hi, On 05/12/18 20:31, Jason Merrill wrote: On 12/5/18 7:45 AM, Paolo Carlini wrote: Hi, as mentioned in one of my last patches, we can now improve this location. Note: in the same function there are a few further issues which I mean to

Re: PR C++/88114 - patch for destructor not generated for "virtual ~destructor() = default"

2018-12-05 Thread Jason Merrill
On 11/21/18 7:19 AM, Tobias Burnus wrote: Hello all, if a class contains any 'virtual ... = 0', it's an abstract class and for an abstract class, the destructor not added to the vtable. For a normal virtual ~class() { } that's not a problem as the class::~class() destructor will be

[Bug c++/88383] New: ICE calling _builtin_has_attribute(r, aligned(N))) on an overaligned reference r

2018-12-05 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88383 Bug ID: 88383 Summary: ICE calling _builtin_has_attribute(r, aligned(N))) on an overaligned reference r Product: gcc Version: 9.0 Status: UNCONFIRMED Severity:

[Bug c++/88362] attribute aligned silently ignored on C++ references

2018-12-05 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88362 --- Comment #7 from Martin Sebor --- I would think for the same reason why attribute aligned applies to the reference and not to the type it refers to. If it makes sense for the following to declare an overaligned reference to an ordinary int

Re: [C++ Patch] Fix grokbitfield location

2018-12-05 Thread Paolo Carlini
Hi, On 05/12/18 20:31, Jason Merrill wrote: On 12/5/18 7:45 AM, Paolo Carlini wrote: Hi, as mentioned in one of my last patches, we can now improve this location. Note: in the same function there are a few further issues which I mean to incrementally fix (eg, the diagnostics for

[Bug c/88382] New: undocumented GNU C extension: C++ raw string literals permitted in GNU C

2018-12-05 Thread richard-gccbugzilla at metafoo dot co.uk
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88382 Bug ID: 88382 Summary: undocumented GNU C extension: C++ raw string literals permitted in GNU C Product: gcc Version: unknown Status: UNCONFIRMED Severity:

Re: Fortran patches

2018-12-05 Thread Fritz Reese
On Wed, Dec 5, 2018 at 12:00 AM Steve Kargl wrote: > > I intend to commit the attached patch on Saturday. Thanks for the work. I assume the patch bootstraps and passes regression tests? RE: > PR fortran/88228 > * expr.c (check_null, check_elemental): Work around -fdec and >

[Ping][Aarch64] v2: Implement Aarch64 SIMD ABI

2018-12-05 Thread Steve Ellcey
Ping. This is a ping of my patch set to implement the SIMD ABI on Aarch64. https://gcc.gnu.org/ml/gcc-patches/2018-11/msg00636.html https://gcc.gnu.org/ml/gcc-patches/2018-11/msg00637.html https://gcc.gnu.org/ml/gcc-patches/2018-11/msg00639.html

Re: [PATCH 1/2] asm qualifiers (PR55681)

2018-12-05 Thread Jason Merrill
On 12/2/18 11:38 AM, Segher Boessenkool wrote: PR55681 observes that currently only one qualifier is allowed for inline asm, so that e.g. "volatile asm" is allowed, "const asm" is also okay (with a warning), but "const volatile asm" gives an error. Also "goto" has to be last. This patch

[Bug middle-end/88381] New: __builtin_thread_pointer missing documentation, defined as implicit_p

2018-12-05 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88381 Bug ID: 88381 Summary: __builtin_thread_pointer missing documentation, defined as implicit_p Product: gcc Version: 9.0 Status: UNCONFIRMED Severity: normal

Re: [PATCH, OpenACC] (1/2) Fix implicit mapping for array slices on lexically-enclosing data constructs (PR70828)

2018-12-05 Thread Julian Brown
On Tue, 4 Dec 2018 15:02:15 +0100 Jakub Jelinek wrote: > On Tue, Aug 28, 2018 at 03:19:19PM -0400, Julian Brown wrote: > > 2018-08-28 Julian Brown > > Cesar Philippidis > > > > PR middle-end/70828 > > > > gcc/ > > * gimplify.c (gimplify_omp_ctx): Add decl_data_clause

[Bug fortran/88364] [9 Regression] Wrong-code due to CLOBBER

2018-12-05 Thread tkoenig at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88364 --- Comment #3 from Thomas Koenig --- Probably easiest to omit the clobber if there is a reference.

[PATCH]: Remove #ifdef PCC_BITFIELD_TYPE_MATTERS check from dwarf2out.c

2018-12-05 Thread Uros Bizjak
This macro is always defined through defaults.h. We can remove #ifdef check from dwarf2out.c 2018-12-05 Uros Bizjak * dwarf2out.c (field_byte_offset): Remove #ifdef PCC_BITFIELD_TYPE_MATTERS check. Bootstrapped and regression tested on x86_64-linux-gnu {,-m32}. OK for mainline?

[Bug c++/88362] attribute aligned silently ignored on C++ references

2018-12-05 Thread joseph at codesourcery dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88362 --- Comment #6 from joseph at codesourcery dot com --- On Wed, 5 Dec 2018, msebor at gcc dot gnu.org wrote: > so that we get consistent behavior for reference members. __alignof__ should > return the corresponding alignment. For example, in

Re: [C++ PATCH] Fix clone_body (PR c++/86669)

2018-12-05 Thread Jakub Jelinek
On Wed, Dec 05, 2018 at 03:49:26PM -0500, Jason Merrill wrote: > On 11/28/18 3:42 AM, Jakub Jelinek wrote: > > Whenever we need to clone a cdtor (either because the target doesn't support > > aliases the way we need, e.g. initlist105.C testcase on darwin, where it has > > been originally reported,

Re: [C++ PATCH] Fix clone_body (PR c++/86669)

2018-12-05 Thread Jason Merrill
On 11/28/18 3:42 AM, Jakub Jelinek wrote: Whenever we need to clone a cdtor (either because the target doesn't support aliases the way we need, e.g. initlist105.C testcase on darwin, where it has been originally reported, or when it has virtual bases, like the made up initlist106.C on

[Bug c++/52869] [DR 1207] "this" not being allowed in noexcept clauses

2018-12-05 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52869 --- Comment #12 from Jonathan Wakely --- The exception specification seems to be processed too early, before the other class members are in scope. This slight variation of comment 5 still fails on trunk: struct S { void g()

[PATCH, i386]: Remove two obsolete/superfluous defines from cygming.h

2018-12-05 Thread Uros Bizjak
GROUP_BITFIELDS_BY_ALIGN was long gone, PCC_BITFIELD_TYPE_MATTERS defaults to 1 in i386.h for all x86 targets. 2018-12-05 Uros Bizjak * config/i386/cygming.h (PCC_BITFIELD_TYPE_MATTERS): Remove. (GROUP_BITFIELDS_BY_ALIGN): Ditto. Tested by building crosscompiler to

[Bug c++/88320] GCC suggests variables that don't exist yet

2018-12-05 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88320 David Malcolm changed: What|Removed |Added Keywords||patch URL|

[Bug testsuite/88332] [9 regression] gcc.dg/Wattributes-10.c fails starting with r265728

2018-12-05 Thread seurer at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88332 seurer at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED

[PATCH] C/C++: don't suggest decls that are being initialized (PR c++/88320)

2018-12-05 Thread David Malcolm
PR c++/88320 reports that the C and C++ FEs can offer bogus suggestions for misspelled identifiers within initializers, where the thing being initialized is suggested. If the user follows these suggestions, it will lead to a -Wuninitialized warning. For example: test.c:9:19: error: 'aresults'

[Bug c++/88380] New: Sequence of not-explicitly initialised, initialised, variable length generates no initialiser

2018-12-05 Thread matt at godbolt dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88380 Bug ID: 88380 Summary: Sequence of not-explicitly initialised, initialised, variable length generates no initialiser Product: gcc Version: 9.0 Status: UNCONFIRMED

[Bug tree-optimization/88064] [9 Regression] Incorrect vectorizer over_widening pattern handling

2018-12-05 Thread rsandifo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88064 rsandifo at gcc dot gnu.org changed: What|Removed |Added Status|ASSIGNED|RESOLVED

[Bug c++/87531] [8/9 Regression] assignment operator does nothing if performed as a call via operator=

2018-12-05 Thread petschy at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87531 petschy at gmail dot com changed: What|Removed |Added CC||petschy at gmail dot com ---

Re: Regarding projects and begineer guide.

2018-12-05 Thread Eric Gallager
On 12/5/18, David Malcolm wrote: > On Wed, 2018-12-05 at 22:48 +0530, Durgesh Kumar wrote: >> Dear sir/mam >> I am durgesh kumar from ABES engineering collenge,ghaziabad,uttar >> pradesh,india. >> >> I am currently in 2nd year of my graduation(Btech CSE ) >> >> I want to contribute in some of

Re: [committed] Handle targets with 2 byte wchar_t better in strlenopt-58.c

2018-12-05 Thread Martin Sebor
On 12/5/18 12:13 PM, Jeff Law wrote: On 12/5/18 11:59 AM, David Edelsohn wrote: Jeff, Thanks for the patch. I continue to see a failure on AIX 32 bit mode (2 byte wchar). FAIL: gcc.dg/strlenopt-58.c scan-tree-dump-times optimized "call_in_true_branch_not_eliminated" 0 I'm not certain if

[Bug c++/87350] NULL-Pointer problem in cplus-dem.c when executing program c++filt

2018-12-05 Thread bernhard.kaindl at thalesgroup dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87350 --- Comment #6 from Bernhard Kaindl --- Created attachment 45167 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45167=edit Safe fix: Before copying work, check if the vectors have been allocated. If not, input wasn't valid. Fixes

[Bug fortran/88357] ICE in parse_associate, at fortran/parse.c:4568

2018-12-05 Thread kargl at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88357 kargl at gcc dot gnu.org changed: What|Removed |Added Priority|P3 |P4

[Bug fortran/88357] ICE in parse_associate, at fortran/parse.c:4568

2018-12-05 Thread gs...@t-online.de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88357 --- Comment #6 from G. Steinmetz --- Sure, this is now pr88379.

[Bug fortran/88379] New: ICE in resolve_assoc_var, at fortran/resolve.c:8750

2018-12-05 Thread gs...@t-online.de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88379 Bug ID: 88379 Summary: ICE in resolve_assoc_var, at fortran/resolve.c:8750 Product: gcc Version: 9.0 Status: UNCONFIRMED Severity: normal Priority: P3

[Bug testsuite/88332] [9 regression] gcc.dg/Wattributes-10.c fails starting with r265728

2018-12-05 Thread pkoning at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88332 --- Comment #11 from pkoning at gcc dot gnu.org --- Thanks, I had forgotten. Seurer, could you update to r265741 or later and check if that cures the issue?

[Bug c++/87350] NULL-Pointer problem in cplus-dem.c when executing program c++filt

2018-12-05 Thread bernhard.kaindl at thalesgroup dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87350 Bernhard Kaindl changed: What|Removed |Added CC||bernhard.kaindl@thalesgroup

[Bug testsuite/88332] [9 regression] gcc.dg/Wattributes-10.c fails starting with r265728

2018-12-05 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88332 --- Comment #10 from Segher Boessenkool --- You have /* { dg-error "alignment of 's' is greater" "" { target pdp11*-*-* } } */ (the "" was added in r265741, the source tree I looked at wasn't up-to-date). dg.exp says this is # dg-error regexp

[Bug c++/85569] [8 Regression] is_invocable(F, decltype(objs)...) fails with "not supported by dump_expr#" unless via indirection

2018-12-05 Thread proski at gnu dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85569 Pavel Roskin changed: What|Removed |Added CC||proski at gnu dot org --- Comment #11

[Bug target/88359] internal compiler error: in push_reload, at reload.c:1360

2018-12-05 Thread geert.linders at hotmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88359 --- Comment #7 from Geert linders --- Hi Pinskia, Which minimum version of avr-gcc is suitable? Cheers - Geert From: pinskia at gcc dot gnu.org Sent: Wednesday, 5 December 2018 10:21 To: geert.lind...@hotmail.com Subject: [Bug

LTO Test Case Help

2018-12-05 Thread Michael Ploujnikov
Hi, I'm trying to write a testcase to reproduce duplicate clone symbols such as in https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88297 I started with a testcase that is known to have constprop clones and split it into two object files: bash-4.2# cat

Re: [C++ Patch] Fix grokbitfield location

2018-12-05 Thread Jason Merrill
On 12/5/18 7:45 AM, Paolo Carlini wrote: Hi, as mentioned in one of my last patches, we can now improve this location. Note: in the same function there are a few further issues which I mean to incrementally fix (eg, the diagnostics for warn_if_not_aligned ICEs for unnamed bit-fields). Tested

[Bug tree-optimization/88372] alloc_size attribute is ignored on function pointers

2018-12-05 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88372 Martin Sebor changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

Re: [committed] Handle targets with 2 byte wchar_t better in strlenopt-58.c

2018-12-05 Thread Jeff Law
On 12/5/18 11:59 AM, David Edelsohn wrote: > Jeff, > > Thanks for the patch. > > I continue to see a failure on AIX 32 bit mode (2 byte wchar). > > FAIL: gcc.dg/strlenopt-58.c scan-tree-dump-times optimized > "call_in_true_branch_not_eliminated" 0 > > I'm not certain if this is AIX-specific or

Re: [committed] Handle targets with 2 byte wchar_t better in strlenopt-58.c

2018-12-05 Thread David Edelsohn
Jeff, Thanks for the patch. I continue to see a failure on AIX 32 bit mode (2 byte wchar). FAIL: gcc.dg/strlenopt-58.c scan-tree-dump-times optimized "call_in_true_branch_not_eliminated" 0 I'm not certain if this is AIX-specific or more fallout from wchar size. I could ignore with target

[Bug target/88359] internal compiler error: in push_reload, at reload.c:1360

2018-12-05 Thread geert.linders at hotmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88359 --- Comment #6 from Geert linders --- Hi Pinskia, Which minimum version of avr-gcc is suitable? Cheers - Geert From: pinskia at gcc dot gnu.org Sent: Wednesday, 5 December 2018 10:21 To:

[Bug fortran/88357] ICE in parse_associate, at fortran/parse.c:4568

2018-12-05 Thread kargl at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88357 --- Comment #5 from kargl at gcc dot gnu.org --- (In reply to G. Steinmetz from comment #4) > Thanks for working on these issues. Gerhard, Can you migrate z2.f90 to its own PR? It is going to require much more effort to fix. My first attempt

[Bug c/88367] [9 Regression] -fno-delete-null-pointer-checks doesn't work properly

2018-12-05 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88367 --- Comment #8 from Jakub Jelinek --- Created attachment 45166 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45166=edit gcc9-pr88367.patch Possible untested patch.

[Bug c++/88378] New: notes for template deduction errors mention "[with U = U]"

2018-12-05 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88378 Bug ID: 88378 Summary: notes for template deduction errors mention "[with U = U]" Product: gcc Version: 9.0 Status: UNCONFIRMED Severity: normal

Re: [PATCH, og8] Add OpenACC 2.6 `acc_get_property' support

2018-12-05 Thread Maciej W. Rozycki
Hi Chung-Lin, > > +module openacc_c_string > > + implicit none > > + > > + interface > > +function strlen (s) bind (C, name = "strlen") > > + use iso_c_binding, only: c_ptr, c_size_t > > + type (c_ptr), intent(in), value :: s > > + integer (c_size_t) :: strlen > > +end

Re: [PATCH, PPC] Fix PR88343.

2018-12-05 Thread Segher Boessenkool
On Wed, Dec 05, 2018 at 05:10:07PM +, Iain Sandoe wrote: > Hi, > > The PR is about unnecessary saves of the pic base register, it shows on m32 > Linux and m32/m64 Darwin. > > The fix is to check that we are in a pic mode and that the picbase has > actually been used. > As a bonus, some

Re: [PATCH/coding style] clarify pointers and operators

2018-12-05 Thread Segher Boessenkool
On Wed, Dec 05, 2018 at 11:37:27AM -0600, Segher Boessenkool wrote: > On Wed, Dec 05, 2018 at 10:04:56AM +, Richard Sandiford wrote: > > Martin Sebor writes: > > > Martin suggested we update the Coding Conventions to describe > > > the expected style for function declarations with a pointer >

Re: [PATCH/coding style] clarify pointers and operators

2018-12-05 Thread Martin Sebor
On 12/5/18 3:04 AM, Richard Sandiford wrote: Thanks for doing this, Martin Sebor writes: Martin suggested we update the Coding Conventions to describe the expected style for function declarations with a pointer return types, and for overloaded operators. Below is the patch. As an aside,

[Bug fortran/88376] [7/8/9 Regression] ICE in is_illegal_recursion, at fortran/resolve.c:1689

2018-12-05 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88376 Dominique d'Humieres changed: What|Removed |Added Priority|P3 |P4

[Bug c++/88362] attribute aligned silently ignored on C++ references

2018-12-05 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88362 --- Comment #5 from Martin Sebor --- I think the attribute on references needs to work the same way as on pointers so that we get consistent behavior for reference members. __alignof__ should return the corresponding alignment. For example, in

[Bug c/87615] Possible excessive compile time with -O2

2018-12-05 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87615 --- Comment #11 from Martin Jambor --- It's actually, ipa_polymorphic_call_context::get_dynamic_type that causes problems here. I'll prepare a patch.

[Bug c++/88146] ice in tsubst_copy, at cp/pt.c:16014

2018-12-05 Thread aoliva at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88146 --- Comment #8 from Alexandre Oliva --- So, no, it's not a copy ctor, but apparently we're reusing the logic that synthesizes them for other non-default ctors.

[Bug c/88367] [9 Regression] -fno-delete-null-pointer-checks doesn't work properly

2018-12-05 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88367 --- Comment #7 from Jakub Jelinek --- In GCC 8 we had: else if (code == POINTER_PLUS_EXPR) { /* For pointer types, we are really only interested in asserting whether the expression evaluates to non-NULL. */

Re: Make claer, when contributions will be ignored

2018-12-05 Thread Joseph Myers
On Wed, 5 Dec 2018, Segher Boessenkool wrote: > Patches are usually ignored because everyone thinks someone else will > handle it. And in this case, it looks like this patch would best be reviewed first in the GDB context - then once committed to binutils-gdb, the committer could post to

Re: [PATCH/coding style] clarify pointers and operators

2018-12-05 Thread Segher Boessenkool
On Wed, Dec 05, 2018 at 10:04:56AM +, Richard Sandiford wrote: > Martin Sebor writes: > > Martin suggested we update the Coding Conventions to describe > > the expected style for function declarations with a pointer > > return types, and for overloaded operators. Below is the patch. > > > >

Re: Regarding projects and begineer guide.

2018-12-05 Thread David Malcolm
On Wed, 2018-12-05 at 22:48 +0530, Durgesh Kumar wrote: > Dear sir/mam > I am durgesh kumar from ABES engineering collenge,ghaziabad,uttar > pradesh,india. > > I am currently in 2nd year of my graduation(Btech CSE ) > > I want to contribute in some of your prestigious project. So i > request you

[Bug c++/88320] GCC suggests variables that don't exist yet

2018-12-05 Thread jg at jguk dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88320 --- Comment #9 from Jonny Grant --- (In reply to David Malcolm from comment #8) > FWIW, clang trunk currently gives the same erroneous suggestion: > > :4:19: error: use of undeclared identifier 'aresults'; did you mean > 'aresult'? > int

[Bug testsuite/88332] [9 regression] gcc.dg/Wattributes-10.c fails starting with r265728

2018-12-05 Thread pkoning at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88332 --- Comment #9 from pkoning at gcc dot gnu.org --- Comment? I thought the comment is the null string after the regexp to match. Should it read { target { pdp11-*-* } } with the extra braces? Other examples show up both with the braces and

[Bug fortran/88377] New: ICE in gfc_omp_clause_copy_ctor, at fortran/trans-openmp.c:614

2018-12-05 Thread gs...@t-online.de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88377 Bug ID: 88377 Summary: ICE in gfc_omp_clause_copy_ctor, at fortran/trans-openmp.c:614 Product: gcc Version: 9.0 Status: UNCONFIRMED Severity: normal

  1   2   3   >