[all-commits] [llvm/llvm-project] ead7a8: [update_cc_test_checks.py] Correctly skip function...

[Alexander Richardson via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: ead7a8beccc18c2cbcde980d2f593267d63c5dc2
      https://github.com/llvm/llvm-project/commit/ead7a8beccc18c2cbcde980d2f593267d63c5dc2
  Author: Alex Richardson <Alexander.Richardson at cl.cam.ac.uk>
  Date:   2020-06-18 (Thu, 18 Jun 2020)

  Changed paths:
    M clang/test/utils/update_cc_test_checks/Inputs/def-and-decl.c
    M clang/test/utils/update_cc_test_checks/Inputs/def-and-decl.c.expected
    M llvm/utils/update_cc_test_checks.py

  Log Message:
  -----------
  [update_cc_test_checks.py] Correctly skip function definitions

Function declarations can in fact have an 'inner' node that lists the
ParmVarDecls. It seems like either the JSON output has changed or that I
tested the original JSON parsing change with test files that only have
function definitions without arguments.

Reviewed By: MaskRay
Differential Revision: https://reviews.llvm.org/D80913


  Commit: 3ce0e92f45f56fc1b93db35dec8c53dceb5c54a5
      https://github.com/llvm/llvm-project/commit/3ce0e92f45f56fc1b93db35dec8c53dceb5c54a5
  Author: Alex Richardson <Alexander.Richardson at cl.cam.ac.uk>
  Date:   2020-06-18 (Thu, 18 Jun 2020)

  Changed paths:
    A clang/test/utils/update_cc_test_checks/Inputs/basic-cplusplus.cpp
    A clang/test/utils/update_cc_test_checks/Inputs/basic-cplusplus.cpp.expected
    A clang/test/utils/update_cc_test_checks/basic-cplusplus.test
    M llvm/utils/update_cc_test_checks.py

  Log Message:
  -----------
  [update_cc_test_checks.py] Handle C++ methods

Previously the script only handled C input, this change extends the JSON
parsing to to also include C++ function types such as methods, constructors
and destructors.

Reviewed By: MaskRay
Differential Revision: https://reviews.llvm.org/D80914


  Commit: 7142fef03d90f4f34e3f6d893f2bc4a671b9e4a5
      https://github.com/llvm/llvm-project/commit/7142fef03d90f4f34e3f6d893f2bc4a671b9e4a5
  Author: Alex Richardson <Alexander.Richardson at cl.cam.ac.uk>
  Date:   2020-06-18 (Thu, 18 Jun 2020)

  Changed paths:
    M libcxx/cmake/Modules/HandleLibCXXABI.cmake

  Log Message:
  -----------
  [libc++] Don't override LIBCXX_CXX_ABI_INCLUDE_PATHS for libcxxrt

When testing libc++ for our cross-compiled CheriBSD target we specify an
explicit LIBCXX_CXX_ABI_INCLUDE_PATHS for libcxxrt. The hardcoded path
/usr/include/c++/v1 was introduced in 61e89737c5daf7b65341936f88f68efb680cdcd4
and overrides any value passed on the CMake command line. Fix this by using
it as a fallback rather than a fixed default value.

Reviewed By: #libc, ldionne
Differential Revision: https://reviews.llvm.org/D82095


Compare: https://github.com/llvm/llvm-project/compare/5fac474fad39...7142fef03d90