diff options
| author | Douglas Gregor <dgregor@apple.com> | 2010-10-08 20:44:28 +0000 |
|---|---|---|
| committer | Douglas Gregor <dgregor@apple.com> | 2010-10-08 20:44:28 +0000 |
| commit | 559c9fb134623e73e8c46d15b15a72fb3d350ec1 (patch) | |
| tree | 926928b1c72c4a9b122ac205294b7330aae91587 /llvm/lib/CodeGen/MachineInstr.cpp | |
| parent | 8ed5b7700860f5a158d6d7577cd9bdc499baa066 (diff) | |
| download | bcm5719-llvm-559c9fb134623e73e8c46d15b15a72fb3d350ec1.tar.gz bcm5719-llvm-559c9fb134623e73e8c46d15b15a72fb3d350ec1.zip | |
This patch fixes multiple issues in clang's designated init builder and
completes support for C1X anonymous struct/union init features:
* Indexed anonymous member initializers should not be expanded. Doing so makes
little sense and would cause unresolvable semantic ambiguity in valid code
(regression introduced by r69153).
* Subobject initialization of (possibly nested) anonymous members are now
referred to with paths relative to the naming record context, eliminating the
synthesis of incorrect implicit InitListExprs that caused CodeGen to assert.
* Field lookup was missing a null check in IdentifierInfo comparison which
caused lookup for a known (already resolved) field to match the first unnamed
data member it encountered leading to silent miscompilation.
* Subobject paths are no longer built using the general purpose
Sema::BuildAnonymousStructUnionMemberPath(). If any corner cases crop up, we
will now assert earlier in Sema instead of passing invalid InitListExprs
through to CodeGen.
Fixes PR6955, from Alp Toker!
llvm-svn: 116098
Diffstat (limited to 'llvm/lib/CodeGen/MachineInstr.cpp')
0 files changed, 0 insertions, 0 deletions
