diff options
author | Eric Fiselier <eric@efcs.ca> | 2018-04-08 06:21:33 +0000 |
---|---|---|
committer | Eric Fiselier <eric@efcs.ca> | 2018-04-08 06:21:33 +0000 |
commit | 4b8c991870a03d6e526610cd20da75d2c96b910f (patch) | |
tree | 594b91926f1d6c42dcdf8bf916f56949e58890af /clang/lib/Sema/TreeTransform.h | |
parent | 80440deed41d88ddae7edca3ae8a56b4dbccc67e (diff) | |
download | bcm5719-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.h | 1 |
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?"); |