public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug middle-end/45579] Re-enable IPA-CP for "fn spec"
       [not found] <bug-45579-4@http.gcc.gnu.org/bugzilla/>
@ 2011-05-12 16:29 ` hubicka at gcc dot gnu.org
  2012-02-02 20:47 ` pinskia at gcc dot gnu.org
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 6+ messages in thread
From: hubicka at gcc dot gnu.org @ 2011-05-12 16:29 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45579

Jan Hubicka <hubicka at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |hubicka at gcc dot gnu.org

--- Comment #1 from Jan Hubicka <hubicka at gcc dot gnu.org> 2011-05-12 15:35:08 UTC ---
Hmm, it should be fairly trivial to update cgraph virtual cloning code to
actually handle those attributes...


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

* [Bug middle-end/45579] Re-enable IPA-CP for "fn spec"
       [not found] <bug-45579-4@http.gcc.gnu.org/bugzilla/>
  2011-05-12 16:29 ` [Bug middle-end/45579] Re-enable IPA-CP for "fn spec" hubicka at gcc dot gnu.org
@ 2012-02-02 20:47 ` pinskia at gcc dot gnu.org
  2012-03-16 23:57 ` steven at gcc dot gnu.org
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 6+ messages in thread
From: pinskia at gcc dot gnu.org @ 2012-02-02 20:47 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45579

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2012-02-02
     Ever Confirmed|0                           |1

--- Comment #2 from Andrew Pinski <pinskia at gcc dot gnu.org> 2012-02-02 20:46:36 UTC ---
Confirmed.


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

* [Bug middle-end/45579] Re-enable IPA-CP for "fn spec"
       [not found] <bug-45579-4@http.gcc.gnu.org/bugzilla/>
  2011-05-12 16:29 ` [Bug middle-end/45579] Re-enable IPA-CP for "fn spec" hubicka at gcc dot gnu.org
  2012-02-02 20:47 ` pinskia at gcc dot gnu.org
@ 2012-03-16 23:57 ` steven at gcc dot gnu.org
  2012-03-19 18:27 ` jamborm at gcc dot gnu.org
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 6+ messages in thread
From: steven at gcc dot gnu.org @ 2012-03-16 23:57 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45579

Steven Bosscher <steven at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |steven at gcc dot gnu.org


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

* [Bug middle-end/45579] Re-enable IPA-CP for "fn spec"
       [not found] <bug-45579-4@http.gcc.gnu.org/bugzilla/>
                   ` (2 preceding siblings ...)
  2012-03-16 23:57 ` steven at gcc dot gnu.org
@ 2012-03-19 18:27 ` jamborm at gcc dot gnu.org
  2012-03-26 15:59 ` jamborm at gcc dot gnu.org
  2013-06-25  9:02 ` dominiq at lps dot ens.fr
  5 siblings, 0 replies; 6+ messages in thread
From: jamborm at gcc dot gnu.org @ 2012-03-19 18:27 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45579

Martin Jambor <jamborm at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jamborm at gcc dot gnu.org

--- Comment #3 from Martin Jambor <jamborm at gcc dot gnu.org> 2012-03-19 18:00:58 UTC ---
Confirmed as recently as this February?  How can that be?  I have just
checked that the testcase from PR 43665, comment #8 and verified the
checkoptimal subroutine is cloned and constants are propagated into
it, despite it having fnspec type attributes.  If you have a testcase
where you believe fnspec attributes are the reason for not performing
IPA-CP, then please post it here, I'd certainly like to have a look.

Having said that, it is certainly true that the new IPA-CP accomplish
this by simply not deleting the newly-unused parameters, whereas
arguably it should update the fnspec attributes instead.  If you think
that is something we should have a PR for, please change the
description accordingly, or open a new PR.

Otherwise I'll close this next week as a WORKSFORME.

(On a related and personal note, I dislike the representation of the
fnspec attributes very much, I think we should move it to the actual
parm_decls or to the call graph or someplace, updating a string_cst
just feels like a work-around.)


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

* [Bug middle-end/45579] Re-enable IPA-CP for "fn spec"
       [not found] <bug-45579-4@http.gcc.gnu.org/bugzilla/>
                   ` (3 preceding siblings ...)
  2012-03-19 18:27 ` jamborm at gcc dot gnu.org
@ 2012-03-26 15:59 ` jamborm at gcc dot gnu.org
  2013-06-25  9:02 ` dominiq at lps dot ens.fr
  5 siblings, 0 replies; 6+ messages in thread
From: jamborm at gcc dot gnu.org @ 2012-03-26 15:59 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45579

Martin Jambor <jamborm at gcc dot gnu.org> changed:

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

--- Comment #4 from Martin Jambor <jamborm at gcc dot gnu.org> 2012-03-26 15:55:04 UTC ---
Fixed with the new IPA-CP.


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

* [Bug middle-end/45579] Re-enable IPA-CP for "fn spec"
       [not found] <bug-45579-4@http.gcc.gnu.org/bugzilla/>
                   ` (4 preceding siblings ...)
  2012-03-26 15:59 ` jamborm at gcc dot gnu.org
@ 2013-06-25  9:02 ` dominiq at lps dot ens.fr
  5 siblings, 0 replies; 6+ messages in thread
From: dominiq at lps dot ens.fr @ 2013-06-25  9:02 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45579

Bug 45579 depends on bug 43665, which changed state.

Bug 43665 Summary: INTENT(IN) etc. optimization of calls: function annotations for noclobber/noescape arguments
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43665

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


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

end of thread, other threads:[~2013-06-25  9:02 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <bug-45579-4@http.gcc.gnu.org/bugzilla/>
2011-05-12 16:29 ` [Bug middle-end/45579] Re-enable IPA-CP for "fn spec" hubicka at gcc dot gnu.org
2012-02-02 20:47 ` pinskia at gcc dot gnu.org
2012-03-16 23:57 ` steven at gcc dot gnu.org
2012-03-19 18:27 ` jamborm at gcc dot gnu.org
2012-03-26 15:59 ` jamborm at gcc dot gnu.org
2013-06-25  9:02 ` dominiq at lps dot ens.fr

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).