[all-commits] [llvm/llvm-project] dcb891: [clang] Fix specialization of non-templated member...

Mariya Podchishchaeva via All-commits all-commits at lists.llvm.org
Mon Jul 24 09:37:22 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: dcb8911316bec81f4f52b40cf0de789f1839730b
      https://github.com/llvm/llvm-project/commit/dcb8911316bec81f4f52b40cf0de789f1839730b
  Author: Podchishchaeva, Mariya <mariya.podchishchaeva at intel.com>
  Date:   2023-07-24 (Mon, 24 Jul 2023)

  Changed paths:
    M clang/docs/ReleaseNotes.rst
    M clang/lib/Sema/SemaTemplateInstantiate.cpp
    A clang/test/SemaTemplate/gh61159.cpp

  Log Message:
  -----------
  [clang] Fix specialization of non-templated member classes of class templates

Explicit specialization doesn't increase depth of template parameters,
so need to be careful when gathering template parameters for
instantiation.
For the case:
```
template<typename T>
struct X {
  struct impl;
};

template <>
struct X<int>::impl {
    template<int ct>
    int f() { return ct; };
};
```
instantiation of `f` used to crash because type template parameter
`int` of explicit specialization was taken into account, but non-type
template parameter `ct` had zero depth and index so wrong parameter
ended up inside of a wrong handler.

Fixes https://github.com/llvm/llvm-project/issues/61159

Reviewed By: aaron.ballman, shafik

Differential Revision: https://reviews.llvm.org/D155705




More information about the All-commits mailing list