[clang] [Clang] Fix UB in #131515 (PR #132091)

@@ -1681,7 +1681,7 @@ std::optional Sema::GetDecompositionElementCount(QualType T, llvm::APSInt TupleSize(Ctx.getTypeSize(Ctx.getSizeType())); switch (isTupleLike(*this, Loc, T, TupleSize)) { case IsTupleLike::Error: -return {}; +return std::nullopt;

[clang] [Clang] Fix UB in #131515 (PR #132091)

llvmbot wrote: @llvm/pr-subscribers-clang Author: cor3ntin (cor3ntin) Changes It turns out trailing objects are uninitialized and APValue assignment operator requires a fully initialized object. --- Full diff: https://github.com/llvm/llvm-project/pull/132091.diff 3 Files Affected: - (m

[clang] [Clang] Fix UB in #131515 (PR #132091)

@@ -1681,7 +1681,7 @@ std::optional Sema::GetDecompositionElementCount(QualType T, llvm::APSInt TupleSize(Ctx.getTypeSize(Ctx.getSizeType())); switch (isTupleLike(*this, Loc, T, TupleSize)) { case IsTupleLike::Error: -return {}; +return std::nullopt;

[clang] [Clang] Fix UB in #131515 (PR #132091)

https://github.com/cor3ntin updated https://github.com/llvm/llvm-project/pull/132091 >From 68ebbf821176dd17a757c32771fa728f21d4dd11 Mon Sep 17 00:00:00 2001 From: Corentin Jabot Date: Wed, 19 Mar 2025 21:05:38 +0100 Subject: [PATCH] [Clang] Fix UB in #131515 It turns out trailing objects are u

[clang] [Clang] Fix UB in #131515 (PR #132091)

https://github.com/fmayer approved this pull request. https://github.com/llvm/llvm-project/pull/132091 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Fix UB in #131515 (PR #132091)

https://github.com/cor3ntin edited https://github.com/llvm/llvm-project/pull/132091 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Fix UB in #131515 (PR #132091)

@@ -1681,7 +1681,7 @@ std::optional Sema::GetDecompositionElementCount(QualType T, llvm::APSInt TupleSize(Ctx.getTypeSize(Ctx.getSizeType())); switch (isTupleLike(*this, Loc, T, TupleSize)) { case IsTupleLike::Error: -return {}; +return std::nullopt;

[clang] [Clang] Fix UB in #131515 (PR #132091)

@@ -1681,7 +1681,7 @@ std::optional Sema::GetDecompositionElementCount(QualType T, llvm::APSInt TupleSize(Ctx.getTypeSize(Ctx.getSizeType())); switch (isTupleLike(*this, Loc, T, TupleSize)) { case IsTupleLike::Error: -return {}; +return std::nullopt;

[clang] [Clang] Fix UB in #131515 (PR #132091)

https://github.com/erichkeane approved this pull request. https://github.com/llvm/llvm-project/pull/132091 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Fix UB in #131515 (PR #132091)

https://github.com/cor3ntin created https://github.com/llvm/llvm-project/pull/132091 It turns out trailing objects are uninitialized and APValue assignment operator requires a fully initialized object. >From e91e16c4129b81c11e16e14f6ed1a379992b870b Mon Sep 17 00:00:00 2001 From: Corentin Jabot