> > The test there is sort of hack, I would just remove it at this stage and > > we can work out better fix for that testcase later. I hope that with my > > plans for declaration merging pass we can get round such weird side > > effects of in place declaration replacement. > > Will do. > > How about all the other uses of DECL_SAVED_TREE in cgraph_analyze_functions? > DECL_SAVED_TREE shouldn't be used past the tree optimizers (??).
There are also few other occurences in cgraph.c and ipa-inline. Those are used to test whether function body is present. When you have body elsewhere now, you can check something else (such as DECL_STRUCT_FUNCTION(cfun)->cfg) We can probably move those changes to mainline to avoid pointers dangling around, I can try to look into this. Honza > > Aldy