Skip to content

Commit 6a571a1

Browse files
authored
[DataLayout] Remove i1 alignment entry (#156657)
I don't think we need to explicitly specify i1 alignment, as this is going to fall back to i8 alignment. This may change behavior if a data layout explicitly sets i8 alignment without also setting i1 layout, but I'd expect this to be a bug fix in that case.
1 parent 193df2a commit 6a571a1

File tree

2 files changed

+0
-2
lines changed

2 files changed

+0
-2
lines changed

llvm/docs/LangRef.rst

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3273,7 +3273,6 @@ specifications are given in this list:
32733273
- ``p[n]:64:64:64`` - Other address spaces are assumed to be the
32743274
same as the default address space.
32753275
- ``S0`` - natural stack alignment is unspecified
3276-
- ``i1:8:8`` - i1 is 8-bit (byte) aligned
32773276
- ``i8:8:8`` - i8 is 8-bit (byte) aligned as mandated
32783277
- ``i16:16:16`` - i16 is 16-bit aligned
32793278
- ``i32:32:32`` - i32 is 32-bit aligned

llvm/lib/IR/DataLayout.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -187,7 +187,6 @@ const char *DataLayout::getManglingComponent(const Triple &T) {
187187
// Default primitive type specifications.
188188
// NOTE: These arrays must be sorted by type bit width.
189189
constexpr DataLayout::PrimitiveSpec DefaultIntSpecs[] = {
190-
{1, Align::Constant<1>(), Align::Constant<1>()}, // i1:8:8
191190
{8, Align::Constant<1>(), Align::Constant<1>()}, // i8:8:8
192191
{16, Align::Constant<2>(), Align::Constant<2>()}, // i16:16:16
193192
{32, Align::Constant<4>(), Align::Constant<4>()}, // i32:32:32

0 commit comments

Comments
 (0)