[all-commits] [llvm/llvm-project] f6ce45: [clang] Correctly(?) handle placeholder types in E...

Quuxplusone via All-commits all-commits at lists.llvm.org
Tue Feb 1 12:16:53 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f6ce456707898f0ae2c70748e896130e1c897960
      https://github.com/llvm/llvm-project/commit/f6ce456707898f0ae2c70748e896130e1c897960
  Author: Arthur O'Dwyer <arthur.j.odwyer at gmail.com>
  Date:   2022-02-01 (Tue, 01 Feb 2022)

  Changed paths:
    M clang/lib/AST/ASTContext.cpp
    M clang/lib/Sema/SemaOverload.cpp
    M clang/lib/Sema/SemaTemplateInstantiate.cpp
    M clang/lib/Sema/TreeTransform.h
    M clang/test/SemaTemplate/constraints.cpp
    A clang/test/SemaTemplate/pr52909.cpp

  Log Message:
  -----------
  [clang] Correctly(?) handle placeholder types in ExprRequirements.

Bug #52905 was originally papered over in a different way, but
I believe this is the actually proper fix, or at least closer to
it. We need to detect placeholder types as close to the front-end
as possible, and cause them to fail constraints, rather than letting
them persist into later stages.

Fixes #52905.
Fixes #52909.
Fixes #53075.

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




More information about the All-commits mailing list