public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug c++/91529] [8/9/10/11 Regression] -fmerge-all-constants leads to corrupt output without inlining
       [not found] <bug-91529-4@http.gcc.gnu.org/bugzilla/>
@ 2020-04-30 18:59 ` jason at gcc dot gnu.org
  2020-05-01 17:59 ` cvs-commit at gcc dot gnu.org
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 6+ messages in thread
From: jason at gcc dot gnu.org @ 2020-04-30 18:59 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91529

Jason Merrill <jason at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
           Assignee|unassigned at gcc dot gnu.org      |jason at gcc dot gnu.org

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug c++/91529] [8/9/10/11 Regression] -fmerge-all-constants leads to corrupt output without inlining
       [not found] <bug-91529-4@http.gcc.gnu.org/bugzilla/>
  2020-04-30 18:59 ` [Bug c++/91529] [8/9/10/11 Regression] -fmerge-all-constants leads to corrupt output without inlining jason at gcc dot gnu.org
@ 2020-05-01 17:59 ` cvs-commit at gcc dot gnu.org
  2020-05-25 22:05 ` cvs-commit at gcc dot gnu.org
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 6+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2020-05-01 17:59 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91529

--- Comment #4 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Jason Merrill <jason@gcc.gnu.org>:

https://gcc.gnu.org/g:a2f32550a085984fbaaec962bf7723514ac71dff

commit r11-16-ga2f32550a085984fbaaec962bf7723514ac71dff
Author: Jason Merrill <jason@redhat.com>
Date:   Fri May 1 13:53:32 2020 -0400

    c++: -fmerge-all-constants vs. destructors [PR91529]

    cp_finish_decl avoids setting TREE_READONLY on TREE_STATIC variables that
    have non-constant construction or destruction, but -fmerge-all-constants
was
    converting an automatic variable to static while leaving TREE_READONLY set.

    Fixed by clearing the flag in cp_finish_decl in the presence of
    -fmerge-all-constants.

    gcc/cp/ChangeLog
    2020-05-01  Jason Merrill  <jason@redhat.com>

            PR c++/91529
            * decl.c (cp_finish_decl): Also clear TREE_READONLY if
            -fmerge-all-constants.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug c++/91529] [8/9/10/11 Regression] -fmerge-all-constants leads to corrupt output without inlining
       [not found] <bug-91529-4@http.gcc.gnu.org/bugzilla/>
  2020-04-30 18:59 ` [Bug c++/91529] [8/9/10/11 Regression] -fmerge-all-constants leads to corrupt output without inlining jason at gcc dot gnu.org
  2020-05-01 17:59 ` cvs-commit at gcc dot gnu.org
@ 2020-05-25 22:05 ` cvs-commit at gcc dot gnu.org
  2020-05-25 22:38 ` cvs-commit at gcc dot gnu.org
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 6+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2020-05-25 22:05 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91529

--- Comment #5 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-10 branch has been updated by Jason Merrill
<jason@gcc.gnu.org>:

https://gcc.gnu.org/g:01ff88c5619897198117b1a460d48774cfcdf396

commit r10-8181-g01ff88c5619897198117b1a460d48774cfcdf396
Author: Jason Merrill <jason@redhat.com>
Date:   Mon May 25 18:04:39 2020 -0400

    c++: -fmerge-all-constants vs. destructors [PR91529]

    cp_finish_decl avoids setting TREE_READONLY on TREE_STATIC variables that
    have non-constant construction or destruction, but -fmerge-all-constants
was
    converting an automatic variable to static while leaving TREE_READONLY set.

    Fixed by clearing the flag in cp_finish_decl in the presence of
    -fmerge-all-constants.

    gcc/cp/ChangeLog
    2020-05-01  Jason Merrill  <jason@redhat.com>

            PR c++/91529
            * decl.c (cp_finish_decl): Also clear TREE_READONLY if
            -fmerge-all-constants.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug c++/91529] [8/9/10/11 Regression] -fmerge-all-constants leads to corrupt output without inlining
       [not found] <bug-91529-4@http.gcc.gnu.org/bugzilla/>
                   ` (2 preceding siblings ...)
  2020-05-25 22:05 ` cvs-commit at gcc dot gnu.org
@ 2020-05-25 22:38 ` cvs-commit at gcc dot gnu.org
  2020-05-25 22:45 ` cvs-commit at gcc dot gnu.org
  2020-05-25 22:47 ` jason at gcc dot gnu.org
  5 siblings, 0 replies; 6+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2020-05-25 22:38 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91529

--- Comment #6 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-9 branch has been updated by Jason Merrill
<jason@gcc.gnu.org>:

https://gcc.gnu.org/g:f76202e096fc0be4db21761399b55f10cd09b20d

commit r9-8621-gf76202e096fc0be4db21761399b55f10cd09b20d
Author: Jason Merrill <jason@redhat.com>
Date:   Mon May 25 18:38:09 2020 -0400

    c++: -fmerge-all-constants vs. destructors [PR91529]

    cp_finish_decl avoids setting TREE_READONLY on TREE_STATIC variables that
    have non-constant construction or destruction, but -fmerge-all-constants
was
    converting an automatic variable to static while leaving TREE_READONLY set.

    Fixed by clearing the flag in cp_finish_decl in the presence of
    -fmerge-all-constants.

    gcc/cp/ChangeLog
    2020-05-01  Jason Merrill  <jason@redhat.com>

            PR c++/91529
            * decl.c (cp_finish_decl): Also clear TREE_READONLY if
            -fmerge-all-constants.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug c++/91529] [8/9/10/11 Regression] -fmerge-all-constants leads to corrupt output without inlining
       [not found] <bug-91529-4@http.gcc.gnu.org/bugzilla/>
                   ` (3 preceding siblings ...)
  2020-05-25 22:38 ` cvs-commit at gcc dot gnu.org
@ 2020-05-25 22:45 ` cvs-commit at gcc dot gnu.org
  2020-05-25 22:47 ` jason at gcc dot gnu.org
  5 siblings, 0 replies; 6+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2020-05-25 22:45 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91529

--- Comment #7 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-8 branch has been updated by Jason Merrill
<jason@gcc.gnu.org>:

https://gcc.gnu.org/g:09831707ce65606f951c97632d882f083ad3ed02

commit r8-10273-g09831707ce65606f951c97632d882f083ad3ed02
Author: Jason Merrill <jason@redhat.com>
Date:   Mon May 25 18:38:09 2020 -0400

    c++: -fmerge-all-constants vs. destructors [PR91529]

    cp_finish_decl avoids setting TREE_READONLY on TREE_STATIC variables that
    have non-constant construction or destruction, but -fmerge-all-constants
was
    converting an automatic variable to static while leaving TREE_READONLY set.

    Fixed by clearing the flag in cp_finish_decl in the presence of
    -fmerge-all-constants.

    gcc/cp/ChangeLog
    2020-05-01  Jason Merrill  <jason@redhat.com>

            PR c++/91529
            * decl.c (cp_finish_decl): Also clear TREE_READONLY if
            -fmerge-all-constants.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug c++/91529] [8/9/10/11 Regression] -fmerge-all-constants leads to corrupt output without inlining
       [not found] <bug-91529-4@http.gcc.gnu.org/bugzilla/>
                   ` (4 preceding siblings ...)
  2020-05-25 22:45 ` cvs-commit at gcc dot gnu.org
@ 2020-05-25 22:47 ` jason at gcc dot gnu.org
  5 siblings, 0 replies; 6+ messages in thread
From: jason at gcc dot gnu.org @ 2020-05-25 22:47 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91529

Jason Merrill <jason at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #8 from Jason Merrill <jason at gcc dot gnu.org> ---
Fixed for 8.5/9.4/10.2/11.

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2020-05-25 22:47 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <bug-91529-4@http.gcc.gnu.org/bugzilla/>
2020-04-30 18:59 ` [Bug c++/91529] [8/9/10/11 Regression] -fmerge-all-constants leads to corrupt output without inlining jason at gcc dot gnu.org
2020-05-01 17:59 ` cvs-commit at gcc dot gnu.org
2020-05-25 22:05 ` cvs-commit at gcc dot gnu.org
2020-05-25 22:38 ` cvs-commit at gcc dot gnu.org
2020-05-25 22:45 ` cvs-commit at gcc dot gnu.org
2020-05-25 22:47 ` jason at gcc dot gnu.org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).