[clang] [clang][Sema] Add regression test for #46331 (PR #226722)

via cfe-commits cfe-commits at lists.llvm.org
Sat Sep 26 11:01:24 PDT 2026


https://github.com/pb01ka created https://github.com/llvm/llvm-project/pull/226722

In this PR, I have added a regression test in `clang/test/SemaCXX/deduced-return-type-cxx14.cpp` for the issue reported in https://github.com/llvm/llvm-project/issues/46331. Here's a complete reasoning behind my change,

- I executed a `git bisect` session between `llvmorg-19-init` and `llvmorg-20-init`. The commands used to initiate the session are as follows,

```sh
git bisect start --term-old=crashes --term-new=fixed
git bisect crashes llvmorg-19-init
git bisect fixed   llvmorg-20-init
```

The final output is as follows,

```sh
eb08c0f1659d12524f58a01bf174177b8acedf2e is the first fixed commit
commit eb08c0f1659d12524f58a01bf174177b8acedf2e
Author: Krystian Stasiowski <sdkrystian at gmail.com>
Date:   Tue Apr 2 08:35:42 2024 -0400

    [Clang][Sema] Fix explicit specializations of member function templates with a deduced return type (#86817)
.
.
.
```

- Upon investigation of https://github.com/llvm/llvm-project/pull/86817/changes/e84e4b6fedb5a643082cd38d0e10d910b299f73f (part of https://github.com/llvm/llvm-project/pull/86817) in accordance with the tracebacks reported in [this](https://github.com/llvm/llvm-project/issues/46331#issue-1077271655) and [this](https://github.com/llvm/llvm-project/issues/46331#issue-1077271655) comment I found that,
  -  Inside `ActOnFunctionDeclarator`, the `Context.getFunctionType(Result, FPT->getParamTypes(), FPT->getExtProtoInfo())` call inside `NewFD->setType(...)` leads to,
  - `return getFunctionTypeInternal(ResultTy, Args, EPI, false);` in `clang/include/clang/AST/ASTContext.h` and hence the crash.
  - Now it is worth noting that the above commit moves the entire `if (getLangOpts().CPlusPlus14 && ...` from `ActOnFunctionDeclarator` block into `CheckFunctionDeclaration`.  Therefore I think the crash got fixed as a side effect of this movement.
  - I observed the presence of `getLangOpts().CPlusPlus14` and hence I tried out the reproducer with clang 18 using both C++11 and C++14 flags,
    - C++11: The crash never happens. The compiler reports the errors and exits. See [here](https://godbolt.org/z/vohjahYTv) (please set C++11 in `Std version` under 🔧 option, the code re-runs for me without this setting whenever I open the link).
    - C++14: The crash does happen. See [here](https://godbolt.org/z/ha5zq5eKW).
- As a consequence of above findings I decided to add the regression test in `clang/test/SemaCXX/deduced-return-type-cxx14.cpp` instead of `clang/test/SemaCXX/trailing-return-0x.cpp` because the latter file only tests on C++11 in which setting the crash would never happen. In addition to this, the fixing commit also added its test in `deduced-return-type-cxx14.cpp` (C++14 onwards enabled).

Please let me know if you find any incorrectness in my investigation/decisions. TY.

Closes https://github.com/llvm/llvm-project/issues/46331

>From 6d29b452dbbcbc7a14d3e0bee9ba23e52b05f221 Mon Sep 17 00:00:00 2001
From: pb01ka <gaganpb08singh at gmail.com>
Date: Sat, 26 Sep 2026 22:45:29 +0530
Subject: [PATCH] [clang] Add regression test for #46331

clang/test/SemaCXX/deduced-return-type-cxx14.cpp was chosen over clang/test/SemaCXX/trailing-return-0x.cpp for adding the regression test because the crash which #46331 reported only happened with C++14 onwards where as trailing-return-0x.cpp is tested only with C++11.
---
 clang/test/SemaCXX/deduced-return-type-cxx14.cpp | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/clang/test/SemaCXX/deduced-return-type-cxx14.cpp b/clang/test/SemaCXX/deduced-return-type-cxx14.cpp
index 10ea9e82f5327..f91335a76feac 100644
--- a/clang/test/SemaCXX/deduced-return-type-cxx14.cpp
+++ b/clang/test/SemaCXX/deduced-return-type-cxx14.cpp
@@ -792,3 +792,10 @@ auto f() {
   return c;
 }
 }
+
+namespace GH46331 {
+template <typename> alignas (enum a{}) // expected-error {{cannot be defined in a type specifier}}
+~a ()()() -> auto; // expected-error {{function with trailing return type must specify return type 'auto', not 'void'}} \
+                   // expected-error {{destructor must be a non-static member function}} \
+                   // expected-error {{destructor cannot be declared as a template}}
+}



More information about the cfe-commits mailing list