[all-commits] [llvm/llvm-project] e6393e: Canonicalize declaration pointers when forming APV...

[Richard Smith via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: e6393ee813178e9d3306b8e3c6949a4f32f8a2cb
      https://github.com/llvm/llvm-project/commit/e6393ee813178e9d3306b8e3c6949a4f32f8a2cb
  Author: Richard Smith <richard at metafoo.co.uk>
  Date:   2020-09-03 (Thu, 03 Sep 2020)

  Changed paths:
    M clang/include/clang/AST/APValue.h
    M clang/lib/AST/APValue.cpp
    M clang/lib/AST/ExprConstant.cpp
    M clang/test/CXX/dcl.dcl/dcl.spec/dcl.constexpr/p9.cpp
    M clang/test/OpenMP/ordered_messages.cpp

  Log Message:
  -----------
  Canonicalize declaration pointers when forming APValues.

References to different declarations of the same entity aren't different
values, so shouldn't have different representations.


  Commit: 052dbe226cb3540c77cf0b3dc4a51a4ab7726b55
      https://github.com/llvm/llvm-project/commit/052dbe226cb3540c77cf0b3dc4a51a4ab7726b55
  Author: Richard Smith <richard at metafoo.co.uk>
  Date:   2020-09-03 (Thu, 03 Sep 2020)

  Changed paths:
    M clang/include/clang/Sema/Sema.h
    M clang/lib/Sema/SemaOverload.cpp

  Log Message:
  -----------
  Remove unused and dangerous overload of PerformImplicitConversion.

Previously we had two overloads where the only real difference beyond
parameter order was whether a reference parameter is const, where one
overload treated the reference parameter as an in-parameter and the
other treated it as an out-parameter!


Compare: https://github.com/llvm/llvm-project/compare/3d54976a7043...052dbe226cb3