Dear Tobias, I think that I have a partial understanding now and will attempt to verify it tonight. Certainly to not build the components, when a derived type is flagged to have proc_pointer components cannot be right just because there can be other components as in the original testcase. This led to the lack of DECL_SIZE information for the field 'term' and the subsequent ICE. The key to understanding is the problem with typebound_operator_9.f03, when the check for proc_pointer components is omitted completely. This implies that there is something missing from the function (sorry, I forget its name) that build proc_pointer fields. I will check that idea - enlightenment might lead to a different patch.
Thanks for the review. Paul On 16 March 2015 at 09:07, Tobias Burnus <tobias.bur...@physik.fu-berlin.de> wrote: > Dear Paul, > > Paul Richard Thomas wrote: >> The ChangeLog says it all. If the check is not done for components >> that are not procedure pointers, >> typebound_operator_9.f03 breaks. I am not entirely sure why this is >> the case but the fix works fine. >> >> Bootstraps and regtests on FC21/x86_64 - OK for 4.8, 4.9 and 5.0? > > OK. I have to admit that I also do not really understand why that's > required. > > Tobias -- Outside of a dog, a book is a man's best friend. Inside of a dog it's too dark to read. Groucho Marx
