[all-commits] [llvm/llvm-project] 6345b0: [C++20] [Modules] Add mangling number for lambda i...

[Chuanqi Xu via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 6345b009c3e58a6cd0eca835d5a935f8784cfda6
      https://github.com/llvm/llvm-project/commit/6345b009c3e58a6cd0eca835d5a935f8784cfda6
  Author: Chuanqi Xu <yedeng.yd at linux.alibaba.com>
  Date:   2025-03-13 (Thu, 13 Mar 2025)

  Changed paths:
    M clang/lib/Sema/SemaLambda.cpp
    A clang/test/Modules/lambda-in-module-purview-2.cppm
    A clang/test/Modules/lambda-in-module-purview.cppm

  Log Message:
  -----------
  [C++20] [Modules] Add mangling number for lambda in non-internal module unit context

Close https://github.com/llvm/llvm-project/issues/59513
Close https://github.com/llvm/llvm-project/issues/110146

As we discussed, this is related to ABI:
https://github.com/itanium-cxx-abi/cxx-abi/issues/186

I was intending to fix this after it gets merged into the ItaniumC++ABI
formally. But it looks like ItaniumC++ABI doesn't update it yet and
there are more issue reports for it.

Luckily Richard had a clear direction guide here though. So I think it
should be good to do this without a formal ItaniumC++ABI wording.

The diff of the patch is slightly larger than it was by a simple
refacoration to simple the control flow a little bit.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications