public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug gcov-profile/60897] Mangled function name in warning during -fprofile-use phase of Firefox build
       [not found] <bug-60897-4@http.gcc.gnu.org/bugzilla/>
@ 2014-04-22  8:53 ` rguenth at gcc dot gnu.org
  2014-04-22  9:34 ` jakub at gcc dot gnu.org
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 6+ messages in thread
From: rguenth at gcc dot gnu.org @ 2014-04-22  8:53 UTC (permalink / raw)
  To: gcc-bugs

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

Richard Biener <rguenth at gcc dot gnu.org> changed:

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

--- Comment #1 from Richard Biener <rguenth at gcc dot gnu.org> ---
.isra.1329

it's a clone.  Why isn't DECL_LANG_SPECIFIC cleared for it?  Thus, why don't
we go the DECL_ABSTRACT_ORIGIN printing way?


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

* [Bug gcov-profile/60897] Mangled function name in warning during -fprofile-use phase of Firefox build
       [not found] <bug-60897-4@http.gcc.gnu.org/bugzilla/>
  2014-04-22  8:53 ` [Bug gcov-profile/60897] Mangled function name in warning during -fprofile-use phase of Firefox build rguenth at gcc dot gnu.org
@ 2014-04-22  9:34 ` jakub at gcc dot gnu.org
  2014-04-22 12:14 ` jamborm at gcc dot gnu.org
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 6+ messages in thread
From: jakub at gcc dot gnu.org @ 2014-04-22  9:34 UTC (permalink / raw)
  To: gcc-bugs

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

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

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

--- Comment #2 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
DECL_LANG_SPECIFIC is only cleared in build_function_decl_skip_args (i.e. when
we are removing or changing some arguments).


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

* [Bug gcov-profile/60897] Mangled function name in warning during -fprofile-use phase of Firefox build
       [not found] <bug-60897-4@http.gcc.gnu.org/bugzilla/>
  2014-04-22  8:53 ` [Bug gcov-profile/60897] Mangled function name in warning during -fprofile-use phase of Firefox build rguenth at gcc dot gnu.org
  2014-04-22  9:34 ` jakub at gcc dot gnu.org
@ 2014-04-22 12:14 ` jamborm at gcc dot gnu.org
  2014-05-14 15:33 ` jamborm at gcc dot gnu.org
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 6+ messages in thread
From: jamborm at gcc dot gnu.org @ 2014-04-22 12:14 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #3 from Martin Jambor <jamborm at gcc dot gnu.org> ---
Right.  IPA-SRA however uses ipa_modify_formal_parameters in ipa-prop
to manipulate parameters which des not reset DECL_LANG_SPECIFIC.  And
it probably should because this means it can currently probably
trigger some variant of PR 60002.

So, if it passes tests, would the following be OK for trunk and the
4.9 branch? 

--- a/gcc/ipa-prop.c
+++ b/gcc/ipa-prop.c
@@ -3650,6 +3650,7 @@ ipa_modify_formal_parameters (tree fndecl,
ipa_parm_adjustment_vec adjustments)

   TREE_TYPE (fndecl) = new_type;
   DECL_VIRTUAL_P (fndecl) = 0;
+  DECL_LANG_SPECIFIC (fndecl) = NULL;
   otypes.release ();
   oparms.release ();
 }


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

* [Bug gcov-profile/60897] Mangled function name in warning during -fprofile-use phase of Firefox build
       [not found] <bug-60897-4@http.gcc.gnu.org/bugzilla/>
                   ` (2 preceding siblings ...)
  2014-04-22 12:14 ` jamborm at gcc dot gnu.org
@ 2014-05-14 15:33 ` jamborm at gcc dot gnu.org
  2014-05-15 12:17 ` jamborm at gcc dot gnu.org
  2014-05-15 12:18 ` jamborm at gcc dot gnu.org
  5 siblings, 0 replies; 6+ messages in thread
From: jamborm at gcc dot gnu.org @ 2014-05-14 15:33 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #4 from Martin Jambor <jamborm at gcc dot gnu.org> ---
Author: jamborm
Date: Wed May 14 15:32:31 2014
New Revision: 210426

URL: http://gcc.gnu.org/viewcvs?rev=210426&root=gcc&view=rev
Log:
2014-05-14  Martin Jambor  <mjambor@suse.cz>

    PR ipa/60897
    * ipa-prop.c (ipa_modify_formal_parameters): Reset DECL_LANG_SPECIFIC.


Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/ipa-prop.c


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

* [Bug gcov-profile/60897] Mangled function name in warning during -fprofile-use phase of Firefox build
       [not found] <bug-60897-4@http.gcc.gnu.org/bugzilla/>
                   ` (3 preceding siblings ...)
  2014-05-14 15:33 ` jamborm at gcc dot gnu.org
@ 2014-05-15 12:17 ` jamborm at gcc dot gnu.org
  2014-05-15 12:18 ` jamborm at gcc dot gnu.org
  5 siblings, 0 replies; 6+ messages in thread
From: jamborm at gcc dot gnu.org @ 2014-05-15 12:17 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #5 from Martin Jambor <jamborm at gcc dot gnu.org> ---
Author: jamborm
Date: Thu May 15 12:16:33 2014
New Revision: 210472

URL: http://gcc.gnu.org/viewcvs?rev=210472&root=gcc&view=rev
Log:
2014-05-15  Martin Jambor  <mjambor@suse.cz>

    PR ipa/60897
    * ipa-prop.c (ipa_modify_formal_parameters): Reset DECL_LANG_SPECIFIC.


Modified:
    branches/gcc-4_9-branch/gcc/ChangeLog
    branches/gcc-4_9-branch/gcc/ipa-prop.c


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

* [Bug gcov-profile/60897] Mangled function name in warning during -fprofile-use phase of Firefox build
       [not found] <bug-60897-4@http.gcc.gnu.org/bugzilla/>
                   ` (4 preceding siblings ...)
  2014-05-15 12:17 ` jamborm at gcc dot gnu.org
@ 2014-05-15 12:18 ` jamborm at gcc dot gnu.org
  5 siblings, 0 replies; 6+ messages in thread
From: jamborm at gcc dot gnu.org @ 2014-05-15 12:18 UTC (permalink / raw)
  To: gcc-bugs

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

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

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

--- Comment #6 from Martin Jambor <jamborm at gcc dot gnu.org> ---
Fixed, at least as far as IPA-SRA clones are concerned.


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

end of thread, other threads:[~2014-05-15 12:18 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <bug-60897-4@http.gcc.gnu.org/bugzilla/>
2014-04-22  8:53 ` [Bug gcov-profile/60897] Mangled function name in warning during -fprofile-use phase of Firefox build rguenth at gcc dot gnu.org
2014-04-22  9:34 ` jakub at gcc dot gnu.org
2014-04-22 12:14 ` jamborm at gcc dot gnu.org
2014-05-14 15:33 ` jamborm at gcc dot gnu.org
2014-05-15 12:17 ` jamborm at gcc dot gnu.org
2014-05-15 12:18 ` jamborm 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).