[all-commits] [llvm/llvm-project] 697ac1: Fix bad mangling of <data-member-prefix> for a clo...

[Richard Smith via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 697ac15a0fc71888c372667bdbc5583ab42d4695
      https://github.com/llvm/llvm-project/commit/697ac15a0fc71888c372667bdbc5583ab42d4695
  Author: Richard Smith <richard at metafoo.co.uk>
  Date:   2021-05-11 (Tue, 11 May 2021)

  Changed paths:
    M clang/include/clang/Basic/CodeGenOptions.h
    M clang/include/clang/Basic/LangOptions.h
    M clang/lib/AST/ItaniumMangle.cpp
    M clang/lib/Frontend/CompilerInvocation.cpp
    M clang/test/CodeGenCXX/clang-abi-compat.cpp
    M clang/test/CodeGenCXX/lambda-expressions-nested-linkage.cpp
    M clang/test/CodeGenCXX/mangle-lambda-explicit-template-params.cpp

  Log Message:
  -----------
  Fix bad mangling of <data-member-prefix> for a closure in the initializer of a variable at global namespace scope.

This implements the direction proposed in
https://github.com/itanium-cxx-abi/cxx-abi/pull/126.

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


  Commit: 3978333b71bff3516ad69aac484b808617976c7a
      https://github.com/llvm/llvm-project/commit/3978333b71bff3516ad69aac484b808617976c7a
  Author: Richard Smith <richard at metafoo.co.uk>
  Date:   2021-05-11 (Tue, 11 May 2021)

  Changed paths:
    M clang/test/SemaTemplate/temp_arg_nontype_cxx20.cpp

  Log Message:
  -----------
  Add test for PR50039.

I believe Clang's behavior is correct according to the standard here,
but this is an unusual situation for which we had no test coverage, so
I'm adding some.


Compare: https://github.com/llvm/llvm-project/compare/6f5ddf67319b...3978333b71bf