[clang] [clang][bytecode] Remove bogus Initializing special case (PR #159933)
via cfe-commits
cfe-commits at lists.llvm.org
Sat Sep 20 09:22:03 PDT 2025
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
More information about the cfe-commits
mailing list