[LLVMdev] IntervalMap - maximum alignment requirements

[James Molloy]

Hi,

 

I'm debugging a fault seen in RuntimeDyldELF on 32-bit machines, stemming
from its use of IntervalMap.

 

The documentation at the top of IntervalMap.h states that it is useful for 4
or 8 byte types, and so RuntimeDyldELF correctly uses it. However, further
down the file is this comment:

 

  // The root data is either a RootLeaf or a RootBranchData instance.

  // We can't put them in a union since C++03 doesn't allow non-trivial

  // constructors in unions.

  // Instead, we use a char array with pointer alignment. The alignment is

  // ensured by the allocator member in the class, but still verified in the

  // constructor. We don't support keys or values that are more aligned than
a

  // pointer.

 

Emphasis mine. So my question is, is IntervalMap supposed to support values
greater than the native pointer width, such as uint64_t on a 32-bit system?

 

Should the documentation be updated to reflect this requirement (and
RuntimeDyldELF rewritten to follow it) or should we fix IntervalMap? (my
current hack fix is to add __attribute__((aligned)) to the end of the
'data[]' member, but I'm sure that's not the best way).

 

Cheers,

 

James
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20120321/140ce8cf/attachment.html>