forked from OSchip/llvm-project
Reapply r134946 with fixes. Tested on Benjamin testcase and other test-suite failures.
llvm-svn: 135091
This commit is contained in:
parent
ad69e65a39
commit
98154a76fd
|
@ -1275,9 +1275,17 @@ void X86_64ABIInfo::classify(QualType Ty, uint64_t OffsetBase,
|
|||
uint64_t Offset = OffsetBase + Layout.getFieldOffset(idx);
|
||||
bool BitField = i->isBitField();
|
||||
|
||||
// AMD64-ABI 3.2.3p2: Rule 1. If ..., or it contains unaligned
|
||||
// fields, it has class MEMORY.
|
||||
// AMD64-ABI 3.2.3p2: Rule 1. If the size of an object is larger than
|
||||
// four eightbytes, or it contains unaligned fields, it has class MEMORY.
|
||||
//
|
||||
// The only case a 256-bit wide vector could be used is when the struct
|
||||
// contains a single 256-bit element. Since Lo and Hi logic isn't extended
|
||||
// to work for sizes wider than 128, early check and fallback to memory.
|
||||
//
|
||||
if (Size > 128 && getContext().getTypeSize(i->getType()) != 256) {
|
||||
Lo = Memory;
|
||||
return;
|
||||
}
|
||||
// Note, skip this test for bit-fields, see below.
|
||||
if (!BitField && Offset % getContext().getTypeAlign(i->getType())) {
|
||||
Lo = Memory;
|
||||
|
|
|
@ -280,7 +280,7 @@ void f39() { f38(x38); f37(x37); }
|
|||
// The two next tests make sure that the struct below is passed
|
||||
// in the same way regardless of avx being used
|
||||
|
||||
// TOBECHECKED: declare void @func40(%struct.t128* byval align 16)
|
||||
// CHECK: declare void @func40(%struct.t128* byval align 16)
|
||||
typedef float __m128 __attribute__ ((__vector_size__ (16)));
|
||||
typedef struct t128 {
|
||||
__m128 m;
|
||||
|
@ -292,7 +292,7 @@ void func41(two128 s) {
|
|||
func40(s);
|
||||
}
|
||||
|
||||
// TOBECHECKED: declare void @func42(%struct.t128_2* byval align 16)
|
||||
// CHECK: declare void @func42(%struct.t128_2* byval align 16)
|
||||
typedef struct xxx {
|
||||
__m128 array[2];
|
||||
} Atwo128;
|
||||
|
|
|
@ -134,3 +134,18 @@ namespace test7 {
|
|||
// CHECK: define void @_ZN5test71zENS_1AES0_S0_S0_S0_NS_12StringDoubleE({{.*}} byval align 8)
|
||||
// CHECK: define void @_ZN5test72zzENS_1AES0_S0_S0_NS_12StringDoubleE({{.*}} i8*
|
||||
}
|
||||
|
||||
namespace test8 {
|
||||
// CHECK: declare void @_ZN5test83fooENS_1BE(%"class.test8::B"* byval align 8)
|
||||
class A {
|
||||
char big[17];
|
||||
};
|
||||
|
||||
class B : public A {};
|
||||
|
||||
void foo(B b);
|
||||
void bar() {
|
||||
B b;
|
||||
foo(b);
|
||||
}
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue