summaryrefslogtreecommitdiffstats
path: root/clang/lib/Sema/TreeTransform.h
diff options
context:
space:
mode:
authorEric Fiselier <eric@efcs.ca>2018-04-08 06:21:33 +0000
committerEric Fiselier <eric@efcs.ca>2018-04-08 06:21:33 +0000
commit4b8c991870a03d6e526610cd20da75d2c96b910f (patch)
tree594b91926f1d6c42dcdf8bf916f56949e58890af /clang/lib/Sema/TreeTransform.h
parent80440deed41d88ddae7edca3ae8a56b4dbccc67e (diff)
downloadbcm5719-llvm-4b8c991870a03d6e526610cd20da75d2c96b910f.tar.gz
bcm5719-llvm-4b8c991870a03d6e526610cd20da75d2c96b910f.zip
[Sema] Fix PR35832 - Ambiguity accessing anonymous struct/union with multiple bases.
Summary: Currently clang doesn't do qualified lookup when building indirect field decl references. This causes ambiguity when the field is in a base class to which there are multiple valid paths even though a qualified name is used. For example: ``` class B { protected: int i; union { int j; }; }; class X : public B { }; class Y : public B { }; class Z : public X, public Y { int a() { return X::i; } // works int b() { return X::j; } // fails }; ``` Reviewers: rsmith, aaron.ballman, rjmccall Reviewed By: rjmccall Subscribers: cfe-commits Differential Revision: https://reviews.llvm.org/D45411 llvm-svn: 329521
Diffstat (limited to 'clang/lib/Sema/TreeTransform.h')
-rw-r--r--clang/lib/Sema/TreeTransform.h1
1 files changed, 0 insertions, 1 deletions
diff --git a/clang/lib/Sema/TreeTransform.h b/clang/lib/Sema/TreeTransform.h
index 1a35c9dbf7c..4de55506223 100644
--- a/clang/lib/Sema/TreeTransform.h
+++ b/clang/lib/Sema/TreeTransform.h
@@ -2239,7 +2239,6 @@ public:
// We have a reference to an unnamed field. This is always the
// base of an anonymous struct/union member access, i.e. the
// field is always of record type.
- assert(!QualifierLoc && "Can't have an unnamed field with a qualifier!");
assert(Member->getType()->isRecordType() &&
"unnamed member not of record type?");
OpenPOWER on IntegriCloud