[PATCH 7/9] Old libiberty fib_heap removed.

Jeff Law law@redhat.com
Fri Nov 14 01:30:00 GMT 2014


On 11/13/14 16:42, Jan Hubicka wrote:
>> gcc/ChangeLog:
>>
>> 2014-11-13  Martin Liska  <mliska@suse.cz>
>>
>> 	* bb-reorder.c (find_traces_1_round): Old fibheap_t type removed.
>> 	* bt-load.c: Include of fibheap.h is removed.
>> 	* cgraphunit.c: Likewise.
>> 	* config/i386/i386.c: Likewise.
>> 	* ipa-inline.c: Likewise.
>> 	* var-tracking.c: Likewise.
>>
>> include/ChangeLog:
>>
>> 2014-11-13  Martin Liska  <mliska@suse.cz>
>>
>> 	* fibheap.h: Remove.
>>
>> libiberty/ChangeLog:
>>
>> 2014-11-13  Martin Liska  <mliska@suse.cz>
>>
>> 	* Makefile.in: Remove.
>> 	* fibheap.c: Remove.
>
> You can not really remove libiberty fibheap, because it is shared across
> projects.  Basically everything in libiberty is set in stone.
> (I believe gdb is other user)
More correctly we'd have to coordinate with other projects prior to 
removing something from libibery.  I think it's OK to just leave the 
libiberty implementation and include API header alone.


>
> Removing the uses from gcc directory is OK of course. From where the fibheap_t is comming
> before this patch is applied?
Right, agreed.
jeff



More information about the Gcc-patches mailing list