This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [lto][patch] Use decl uid instead of node uid in ipa-pure-const


> Ah, but it should not change nodes ids, it is simply resusing dead IDs
> for new nodes. If it changes IDs on LTO, then something is wrong.

I am sorry that I don't remember the details, but I think what was
happening was something like

* A node with id 42 is created
* We (LTO) remember something about the node
* That node is removed
* A new node with id 42 is created
* LTO gets confused

As Diego noticed, the clone infrastructure changed and the patch might
not be necessary any more. What I would suggest is to revert it a tree
and see if libgomp.c++/for-4.C still works. If it does, just remove
the patch for real and continue with the merge.

Cheers,
-- 
Rafael Avila de Espindola

Google | Gordon House | Barrow Street | Dublin 4 | Ireland
Registered in Dublin, Ireland | Registration Number: 368047


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]