diff options
author | Douglas Gregor <dgregor@apple.com> | 2010-05-22 05:17:18 +0000 |
---|---|---|
committer | Douglas Gregor <dgregor@apple.com> | 2010-05-22 05:17:18 +0000 |
commit | aae38d66101b9f6642348a3a579407a5b3f012ad (patch) | |
tree | f2a87abdcb3f948ef984bb5e1c992c33edcc72db /clang/test/CodeGenCXX/c99-variable-length-array.cpp | |
parent | 987a1576d283cce924d156fe6ae78f3f3de0b7c8 (diff) | |
download | bcm5719-llvm-aae38d66101b9f6642348a3a579407a5b3f012ad.tar.gz bcm5719-llvm-aae38d66101b9f6642348a3a579407a5b3f012ad.zip |
Improve our handling of reference binding for subobjects of
temporaries. There are actually several interrelated fixes here:
- When converting an object to a base class, it's only an lvalue
cast when the original object was an lvalue and we aren't casting
pointer-to-derived to pointer-to-base. Previously, we were
misclassifying derived-to-base casts of class rvalues as lvalues,
causing various oddities (including problems with reference binding
not extending the lifetimes of some temporaries).
- Teach the code for emitting a reference binding how to look
through no-op casts and parentheses directly, since
Expr::IgnoreParenNoOpCasts is just plain wrong for this. Also, make
sure that we properly look through multiple levels of indirection
from the temporary object, but destroy the actual temporary object;
this fixes the reference-binding issue mentioned above.
- Teach Objective-C message sends to bind the result as a temporary
when needed. This is actually John's change, but it triggered the
reference-binding problem above, so it's included here. Now John
can actually test his return-slot improvements.
llvm-svn: 104434
Diffstat (limited to 'clang/test/CodeGenCXX/c99-variable-length-array.cpp')
0 files changed, 0 insertions, 0 deletions