[llvm-commits] [llvm] r94738 - in /llvm/trunk: include/llvm/Constants.h lib/VMCore/Constants.cpp

Duncan Sands baldrick at free.fr
Thu Jan 28 01:32:45 PST 2010


Hi Dan,

> Make getAlignOf return an i64, for consistency with getSizeOf and
> getOffsetOf, and remove the comment about assuming i8 is byte-aligned,
> which is no longer applicable.

strictly speaking the algorithm doesn't require that the "aligning type" (which
seems to be i1 these days) has alignment 1, what it requires is that no type has
a smaller alignment than it.  For example, it would work if i1 had an alignment
of 2 as long as all types have alignment at least 2.

Ciao,

Duncan.



More information about the llvm-commits mailing list