[all-commits] [llvm/llvm-project] 675a55: Remove redundant LLVM_HAS_RVALUE_REFERENCE_THIS an...

[Simon Pilgrim via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 675a557316ef80f57b96c13a77201d63681d81d2
      https://github.com/llvm/llvm-project/commit/675a557316ef80f57b96c13a77201d63681d81d2
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2022-02-02 (Wed, 02 Feb 2022)

  Changed paths:
    M clang/include/clang/ASTMatchers/ASTMatchersInternal.h
    M clang/include/clang/Basic/DirectoryEntry.h
    M clang/include/clang/StaticAnalyzer/Core/PathSensitive/ExplodedGraph.h
    M llvm/include/llvm/ADT/Optional.h
    M llvm/include/llvm/ADT/PointerIntPair.h
    M llvm/include/llvm/Support/Compiler.h
    M llvm/unittests/ADT/OptionalTest.cpp

  Log Message:
  -----------
  Remove redundant LLVM_HAS_RVALUE_REFERENCE_THIS and LLVM_LVALUE_FUNCTION defines

Now that VS2017 support has been dropped (D114639), the LLVM_HAS_RVALUE_REFERENCE_THIS define is always true and the LLVM_LVALUE_FUNCTION define is always enabled for ref-qualifiers.

This patch proposes we remove the defines and use the qualifiers directly.

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