[Bug c/22065] -fdump-tree-original causes static function to emitted with the same name

[Bug c/22065] -fdump-tree-original causes static function to emitted with the same name

[pinskia at gcc dot gnu dot org]

------- Comment #2 from pinskia at gcc dot gnu dot org  2005-10-16 03:48 -------
Hmm, the only problem with calling DECL_NAME I think is with C++ code but I
have to double check on that.


-- 


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

[Bug c/22065] -fdump-tree-original causes static function to emitted with the same name

[pinskia at gcc dot gnu dot org]

------- Comment #3 from pinskia at gcc dot gnu dot org  2005-10-16 03:50 -------
Yes, with the C++ dumps, we would like the assemble name.
Maybe we are calling c_genericize too early.

I am going to look into this more.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |pinskia at gcc dot gnu dot
                   |dot org                     |org
             Status|NEW                         |ASSIGNED


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

[Bug c/22065] -fdump-tree-original causes static function to emitted with the same name

[pinskia at gcc dot gnu dot org]

------- Comment #4 from pinskia at gcc dot gnu dot org  2005-10-16 04:27 -------
Created an attachment (id=9999)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=9999&action=view)
Patch which should fix it but needs testing

This patch moves where we do the finalizing of the function in unit at a time
mode from when we finish a function to when we pop the file scope.  This will
fix the -fdump-tree-orginal problem but I don't know if it works in all cases. 
I have not done a bootstrap/test on this yet but I will be doing that tonight
with also a --enable-intermodule build with -fdump-tree-all-vops so we get the
code really tested.


-- 


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

[Bug c/22065] -fdump-tree-original causes static function to emitted with the same name

[pinskia at gcc dot gnu dot org]

------- Comment #5 from pinskia at gcc dot gnu dot org  2005-10-16 16:18 -------
(In reply to comment #4)
> Created an attachment (id=9999)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=9999&action=view) [edit]
> Patch which should fix it but needs testing

Note this patch does not work.  We are trying to access an already free block. 
I have not looked into it yet but the short testcase is:
extern inline int f(void){}


-- 


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

[Bug c/22065] -fdump-tree-original causes static function to emitted with the same name

[pinskia at gcc dot gnu dot org]

------- Comment #6 from pinskia at gcc dot gnu dot org  2005-10-16 16:45 -------
(In reply to comment #5)
> Note this patch does not work.  We are trying to access an already free block. 
> I have not looked into it yet but the short testcase is:
> extern inline int f(void){}

I know why now, c_genericize or cgraph_finalize_function calls ggc_collect and
the new block is not going to be marked as used.


-- 


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

[Bug c/22065] -fdump-tree-original causes static function to emitted with the same name

[pinskia at gcc dot gnu dot org]

------- Comment #7 from pinskia at gcc dot gnu dot org  2005-10-16 17:01 -------
I have a new patch which I am testing right now.  The only change between this
patch and the one attached is an additional static variable to make sure that
block is not GC'd.  Once finish testing I will post it.


-- 


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

[Bug c/22065] -fdump-tree-original causes static function to emitted with the same name

[pinskia at gcc dot gnu dot org]

------- Comment #8 from pinskia at gcc dot gnu dot org  2005-10-16 18:52 -------
The patch still does not work but here is the reduced testcase:
      typedef long unsigned int size_t;
     void * __wrap_malloc (size_t c) __attribute__ (( alias ("malloc") ));

This on x86_64-pc-linux-gnu, I will still look into it.


-- 


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

[Bug c/22065] -fdump-tree-original causes static function to emitted with the same name

[pinskia at gcc dot gnu dot org]

------- Comment #9 from pinskia at gcc dot gnu dot org  2005-10-23 00:04 -------
No longer working on this, laptop is dead.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|pinskia at gcc dot gnu dot  |unassigned at gcc dot gnu
                   |org                         |dot org
             Status|ASSIGNED                    |NEW


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

[Bug c/22065] -fdump-tree-original causes static function to emitted with the same name

[belyshev at depni dot sinp.msu.ru]

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

Serge Belyshev <belyshev at depni dot sinp.msu.ru> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |belyshev at depni dot
                   |                            |sinp.msu.ru
         Resolution|                            |FIXED
      Known to fail|                            |

--- Comment #10 from Serge Belyshev <belyshev at depni dot sinp.msu.ru> 2011-11-07 19:31:45 UTC ---
Fixed at some point between 4.2 and 4.3.

[Bug c/22065] -fdump-tree-original causes static function to emitted with the same name

[belyshev at depni dot sinp dot msu dot ru]

------- Additional Comments From belyshev at depni dot sinp dot msu dot ru  2005-06-17 10:24 -------
Confirmed.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|                            |1
      Known to fail|                            |4.0.0 4.0.1 4.1.0
   Last reconfirmed|0000-00-00 00:00:00         |2005-06-17 10:24:54
               date|                            |


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