This is the mail archive of the binutils@sourceware.org mailing list for the binutils 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][gold] Don't put rodata and text in the same segment.


Rafael Espindola <espindola@google.com> writes:

> Looking a bit more in the code I was curious why the optimization of
> trying to keep the data contiguous on disk but not in memory could not
> be applied in more cases. I also noticed that was_readonly was never
> being set back to false. Is it because the code knows how the segments
> are ordered? That is wrong in the presence of large segments, no?
>
> Anyway, I tried to apply the optimization unconditionally and I could
> not find a case that fails. Would you mind reviewing the attached
> patch first? If you do know a case that would fail let me know and I
> will add a testcase for it instead.
>
> 2010-09-03  Rafael Espindola  <espindola@google.com>
>
>        * layout.cc (Layout::set_segment_offsets): Try harder to save disk
> 	space.

As noted in my last reply, I think this is correct whenever the segment
flags change.  When the segment flags do not change there is no reason
to do this.

Ian


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