maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Don't put too much thought into whether or not to capture a 

type-dependent intermediate result in a postfix ++ pseudo-
object operation.

Test case by Tong Shen.

llvm-svn: 179637
This commit is contained in:
John McCall 2013-04-16 22:32:04 +00:00
parent 095c58f1c4
commit 0d9dd73847
2 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
clang/lib/Sema/SemaPseudoObject.cpp
View File

@ -441,7 +441,8 @@ PseudoOpBuilder::buildIncDecOperation(Scope *Sc, SourceLocation opcLoc,
QualType resultType = result.get()->getType();
// That's the postfix result.
if (UnaryOperator::isPostfix(opcode) && CanCaptureValueOfType(resultType)) {
if (UnaryOperator::isPostfix(opcode) &&
(result.get()->isTypeDependent() || CanCaptureValueOfType(resultType))) {
result = capture(result.take());
setResultToLastSemantic();
}

9
clang/test/SemaCXX/MicrosoftExtensions.cpp
View File

@ -302,13 +302,14 @@ struct SP9 {
T GetV() { return 0; }
void SetV(T v) {}
void f() { V = this->V; V < this->V; }
//void g() { V++; }
//void h() { V*=2; }
void g() { V++; }
void h() { V*=2; }
};
struct SP10 {
SP10(int v) {}
bool operator<(const SP10& v) { return true; }
SP10 operator*(int v) { return *this; }
SP10 operator+(int v) { return *this; }
SP10& operator=(const SP10& v) { return *this; }
};
void TestSP9() {
@ -329,6 +330,6 @@ void TestSP9() {
SP9<SP10> c3;
c3.f(); // Overloaded binary op operand
//c3.g(); // Overloaded incdec op operand
//c3.h(); // Overloaded unary op operand
c3.g(); // Overloaded incdec op operand
c3.h(); // Overloaded unary op operand
}