llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: Timm Baeder (tbaederr) <details> <summary>Changes</summary> This doesn't seem to be needed anymore and causes problems. Fixes #<!-- -->159787 --- Full diff: https://github.com/llvm/llvm-project/pull/159933.diff 2 Files Affected: - (modified) clang/lib/AST/ByteCode/Compiler.cpp (+2-7) - (modified) clang/test/AST/ByteCode/cxx03.cpp (+6) ``````````diff diff --git a/clang/lib/AST/ByteCode/Compiler.cpp b/clang/lib/AST/ByteCode/Compiler.cpp index fafec47f7de3c..cfc373724dfd7 100644 --- a/clang/lib/AST/ByteCode/Compiler.cpp +++ b/clang/lib/AST/ByteCode/Compiler.cpp @@ -2378,13 +2378,8 @@ bool Compiler<Emitter>::VisitMemberExpr(const MemberExpr *E) { return this->visitDeclRef(Member, E); } - if (Initializing) { - if (!this->delegate(Base)) - return false; - } else { - if (!this->visit(Base)) - return false; - } + if (!this->visit(Base)) + return false; // Base above gives us a pointer on the stack. const auto *FD = cast<FieldDecl>(Member); diff --git a/clang/test/AST/ByteCode/cxx03.cpp b/clang/test/AST/ByteCode/cxx03.cpp index 10e5232b9f873..58d7f3632082d 100644 --- a/clang/test/AST/ByteCode/cxx03.cpp +++ b/clang/test/AST/ByteCode/cxx03.cpp @@ -40,3 +40,9 @@ struct B2 : B { }; _Static_assert(&(B2().a) == &p, ""); // both-error {{taking the address of a temporary object of type 'int'}} \ // both-error {{not an integral constant expression}} + +typedef __attribute__((ext_vector_type(4))) int vi4b; +struct S { + vi4b w; +}; +const int s = S().w[1]; `````````` </details> https://github.com/llvm/llvm-project/pull/159933 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
