2009-12-16 04:14:24 +08:00
|
|
|
// RUN: %clang_cc1 -fsyntax-only -verify %s -Winvalid-offsetof
|
2009-05-02 07:20:30 +08:00
|
|
|
|
|
|
|
struct NonPOD {
|
|
|
|
virtual void f();
|
|
|
|
int m;
|
|
|
|
};
|
|
|
|
|
|
|
|
struct P {
|
|
|
|
NonPOD fieldThatPointsToANonPODType;
|
|
|
|
};
|
|
|
|
|
|
|
|
void f() {
|
2010-03-10 19:27:22 +08:00
|
|
|
int i = __builtin_offsetof(P, fieldThatPointsToANonPODType.m); // expected-warning{{offset of on non-POD type 'P'}}
|
2009-05-02 07:20:30 +08:00
|
|
|
}
|
|
|
|
|
2009-12-04 15:18:51 +08:00
|
|
|
struct Base { int x; };
|
|
|
|
struct Derived : Base { int y; };
|
|
|
|
int o = __builtin_offsetof(Derived, x); // expected-warning{{offset of on non-POD type}}
|
2009-12-12 15:25:49 +08:00
|
|
|
|
|
|
|
const int o2 = sizeof(__builtin_offsetof(Derived, x));
|
Completely reimplement __builtin_offsetof, based on a patch by Roberto
Amadini.
This change introduces a new expression node type, OffsetOfExpr, that
describes __builtin_offsetof. Previously, __builtin_offsetof was
implemented using a unary operator whose subexpression involved
various synthesized array-subscript and member-reference expressions,
which was ugly and made it very hard to instantiate as a
template. OffsetOfExpr represents the AST more faithfully, with proper
type source information and a more compact representation.
OffsetOfExpr also has support for dependent __builtin_offsetof
expressions; it can be value-dependent, but will never be
type-dependent (like sizeof or alignof). This commit introduces
template instantiation for __builtin_offsetof as well.
There are two major caveats to this patch:
1) CodeGen cannot handle the case where __builtin_offsetof is not a
constant expression, so it produces an error. So, to avoid
regressing in C, we retain the old UnaryOperator-based
__builtin_offsetof implementation in C while using the shiny new
OffsetOfExpr implementation in C++. The old implementation can go
away once we have proper CodeGen support for this case, which we
expect won't cause much trouble in C++.
2) __builtin_offsetof doesn't work well with non-POD class types,
particularly when the designated field is found within a base
class. I will address this in a subsequent patch.
Fixes PR5880 and a bunch of assertions when building Boost.Python
tests.
llvm-svn: 102542
2010-04-29 06:16:22 +08:00
|
|
|
|
|
|
|
struct HasArray {
|
|
|
|
int array[17];
|
|
|
|
};
|
|
|
|
|
|
|
|
// Constant and non-constant offsetof expressions
|
|
|
|
void test_ice(int i) {
|
|
|
|
int array0[__builtin_offsetof(HasArray, array[5])];
|
|
|
|
int array1[__builtin_offsetof(HasArray, array[i])]; // expected-error{{variable length arrays are not permitted in C++}}
|
|
|
|
}
|
2010-04-29 06:36:06 +08:00
|
|
|
|
|
|
|
// Bitfields
|
|
|
|
struct has_bitfields {
|
|
|
|
int i : 7;
|
|
|
|
int j : 12; // expected-note{{bit-field is declared here}}
|
|
|
|
};
|
|
|
|
|
|
|
|
int test3 = __builtin_offsetof(struct has_bitfields, j); // expected-error{{cannot compute offset of bit-field 'j'}}
|