[all-commits] [llvm/llvm-project] 99daf6: [C++20] [Modules] Don't handle no linkage entities...

[Chuanqi Xu via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 99daf6b2636245446cab0a7800e36fd34777e599
      https://github.com/llvm/llvm-project/commit/99daf6b2636245446cab0a7800e36fd34777e599
  Author: Chuanqi Xu <yedeng.yd at linux.alibaba.com>
  Date:   2022-07-26 (Tue, 26 Jul 2022)

  Changed paths:
    M clang/lib/Sema/SemaOverload.cpp
    A clang/test/Modules/ctor.arg.dep.cppm

  Log Message:
  -----------
  [C++20] [Modules] Don't handle no linkage entities when overloading

The original implementation uses `ND->getFormalLinkage() <=
Linkage::InternalLinkage`. It is not right since the spec only says
internal linkage and it doesn't mention 'no linkage'. This matters when
we consider constructors. According to [class.ctor.general]p1,
constructors have no name so constructors have no linkage too.