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

via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 19 13:45:55 PDT 2025


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

Yes, it should not have, as was pointed out in the original PR (and it's clearly bogus).
I edited the commit message to clarify.
Thanks!

https://github.com/llvm/llvm-project/pull/132091


More information about the cfe-commits mailing list