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: [PATCH] ada: Fix crash with deeply inherited function


>     gcc/ada/
> 	PR ada/34149
> 	* exp_ch3.adb (Make_Controlling_Function_Wrappers): Ignore
> 	primitive operations that have just been created and not yet
> 	analyzed.

Here is Ed's review of your change:

There is an order-of-elaboration problem here, but the suggested fix does not
seem to be correct. It disables the creation of function wrappers for
inherited functions of null extensions in some casess that crash the front-end,
but the patch does not create those wrappers at all. The proper fix would
require that we find the original function that is being inherited multiple
times, and create a wrapper for it.

Arno


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