[all-commits] [llvm/llvm-project] c99841: [clang][test] add TestLanguage.def to specify all ...

Julian Schmidt via All-commits all-commits at lists.llvm.org
Sat Aug 24 04:02:18 PDT 2024


  Branch: refs/heads/users/5chmidti/rm_not_needed_run_overload_in_BoundNodesCallback
  Home:   https://github.com/llvm/llvm-project
  Commit: c99841652e88b96b7d339a7716623554352bd76d
      https://github.com/llvm/llvm-project/commit/c99841652e88b96b7d339a7716623554352bd76d
  Author: Julian Schmidt <git.julian.schmidt at gmail.com>
  Date:   2024-08-24 (Sat, 24 Aug 2024)

  Changed paths:
    M clang/include/clang/Testing/CommandLineArgs.h
    M clang/include/clang/Testing/TestClangConfig.h
    A clang/include/clang/Testing/TestLanguage.def
    M clang/lib/Testing/CommandLineArgs.cpp
    M clang/unittests/AST/MatchVerifier.h
    M clang/unittests/ASTMatchers/ASTMatchersNodeTest.cpp
    M clang/unittests/ASTMatchers/ASTMatchersTraversalTest.cpp
    M clang/unittests/Tooling/Syntax/BuildTreeTest.cpp
    M clang/unittests/Tooling/Syntax/MutationsTest.cpp
    M clang/unittests/Tooling/Syntax/SynthesisTest.cpp
    M clang/unittests/Tooling/Syntax/TreeTest.cpp
    M clang/unittests/Tooling/Syntax/TreeTestBase.cpp

  Log Message:
  -----------
  [clang][test] add TestLanguage.def to specify all tested language versions

Adds a def file to have a single location where tested language versions are
specified. Removes the need to update multiple locations in the testing
infrastructure to add a new language version to be tested.
Test instatiation can now include all languages without needing to
specify them.
This patch also adds pretty printing for instantiated test names.
That means, that a test instantiated with C++23 will have the
name `...TestSuite/TestName/CXX23` instead ending with some number
(index of the argument for instantiation of the test), which improves a
better experience when encountering a test failure with a specific
language version. The suffix will also contain an `_win` if the target
contains `win`.


  Commit: c1766e0d4936349629a0e81f729c996dde46933e
      https://github.com/llvm/llvm-project/commit/c1766e0d4936349629a0e81f729c996dde46933e
  Author: Julian Schmidt <git.julian.schmidt at gmail.com>
  Date:   2024-08-24 (Sat, 24 Aug 2024)

  Changed paths:
    M clang/include/clang/Testing/TestLanguage.def

  Log Message:
  -----------
  add macro description


  Commit: 2f237d3a1cf8911eb3f1ee08a2346426587d8d49
      https://github.com/llvm/llvm-project/commit/2f237d3a1cf8911eb3f1ee08a2346426587d8d49
  Author: Julian Schmidt <git.julian.schmidt at gmail.com>
  Date:   2024-08-24 (Sat, 24 Aug 2024)

  Changed paths:
    M clang/include/clang/Testing/TestLanguage.def

  Log Message:
  -----------
  add c2y


  Commit: 1f1503d726ad611ed82780416d4148609d0d15fa
      https://github.com/llvm/llvm-project/commit/1f1503d726ad611ed82780416d4148609d0d15fa
  Author: Julian Schmidt <git.julian.schmidt at gmail.com>
  Date:   2024-08-24 (Sat, 24 Aug 2024)

  Changed paths:
    M clang/unittests/ASTMatchers/ASTMatchersNodeTest.cpp
    M clang/unittests/ASTMatchers/ASTMatchersTest.h
    M clang/unittests/ASTMatchers/ASTMatchersTraversalTest.cpp

  Log Message:
  -----------
  [clang][test] remove unused `run` overload in `BoundNodesCallback`

The overload that did not take the additional `ASTContext *` argument is
unnecessary when the context could simply be commented out, as it is
always passed to `run` from `VerifyMatcher::run`.
This patch removes the single-argument overload in favor of having a
single overload.


Compare: https://github.com/llvm/llvm-project/compare/10b51f4df6b5...1f1503d726ad

To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list