lkml.org 
[lkml]   [2009]   [Jan]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [patch 2/8] compiler-gcc.h: add more comments to RELOC_HIDE
Christoph Lameter wrote:
> On Thu, 15 Jan 2009, Rusty Russell wrote:
>
>>> The cast should cause the C compiler to drop all assumptions about size.
>> No, and that's the point. Sorry, at this point you need to talk to a gcc expert. As I have said, I did and I believe what he told me.
>
> The gcc expert that created this measss is cced on this thread and so
> far he not spoken up. Richard?

It has been a long time, and I don't recall all of the assumptions
involved from the time.

It was probably a combination of object size assumptions, as well as
problems with relocations. Stuff like "int foo" is known to be
allocated within the small data structure, and thus various types of
small-data-section relocations are valid for it. Then we do stuff like
"(void *)&foo - large_constant" which don't work with those sorts of
relocations.

I didn't explore the space of possible solutions, merely gave Rusty a
solution that I knew would work, and would never fail because the
compiler would never look through the asm.

I wouldn't be surprised if the compiler thought "(long)&foo -
large_constant" could be put back together into a small-data relocation,
simply because at the level at which that optimization is performed,
we've thrown away type data like long and void*; we only have modes.

Why are you wanting to change this at all? It works as it is.


r~


\
 
 \ /
  Last update: 2009-01-16 00:17    [W:0.081 / U:0.092 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site