[all-commits] [llvm/llvm-project] 1db60c: Remove redundant check for access in the conversio...

[Richard Smith via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 1db60c1307ac2e24796047c39d09bf400c22e531
      https://github.com/llvm/llvm-project/commit/1db60c1307ac2e24796047c39d09bf400c22e531
  Author: Richard Smith <richard at metafoo.co.uk>
  Date:   2020-11-29 (Sun, 29 Nov 2020)

  Changed paths:
    M clang/lib/Sema/SemaExpr.cpp
    M clang/test/CXX/class.access/class.access.base/p1.cpp
    M clang/test/CXX/class.access/class.access.base/p5.cpp
    M clang/test/CXX/class.access/class.friend/p1.cpp
    M clang/test/CXX/class.access/class.protected/p1.cpp
    M clang/test/CXX/class.access/p4.cpp
    M clang/test/CXX/drs/dr0xx.cpp
    M clang/test/CXX/drs/dr1xx.cpp
    M clang/test/CXX/drs/dr2xx.cpp
    M clang/test/CXX/drs/dr3xx.cpp
    M clang/test/SemaCXX/anonymous-union.cpp
    M clang/test/SemaCXX/conversion-function.cpp

  Log Message:
  -----------
  Remove redundant check for access in the conversion from the naming
class to the declaring class in a class member access.

This check does not appear to be backed by any rule in the standard (the
rule in question was likely removed over the years), and only ever
produces duplicate diagnostics. (It's also not meaningful because there
isn't a unique declaring class after the resolution of core issue 39.)