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] Remove after_inlining flag


On Thu, Oct 16, 2008 at 11:36, Rafael Espindola <espindola@google.com> wrote:

> The patch is for lto. Debug is completely broken already.

Sure.  However, the question here is whether this is something that
can be addressed with the gimple types.  If not, we will need to keep
this scheme or something similar.


>> No, replacing it by 1 will lead to divergence on -g and ICEs on weird
>> C++ testcases.  Replacing it with 0 will lead to missed optimizations.
>> Just keep it 1 until after last inlining happens.
>
> For lto  I think the best option now is to take the missed optimizations.

Yes.  So, keep after_inlining to 0 during cc1 and only set it in lto1?
 Jan, you seem to be proposing the opposite.


Diego.


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