[all-commits] [llvm/llvm-project] 025641: [libc++] NFC: Minor cleanups in config.py

Louis Dionne via All-commits all-commits at lists.llvm.org
Tue Apr 14 05:28:29 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 025641342b952b699684aeaee4a383af83f18d4d
      https://github.com/llvm/llvm-project/commit/025641342b952b699684aeaee4a383af83f18d4d
  Author: Louis Dionne <ldionne at apple.com>
  Date:   2020-04-14 (Tue, 14 Apr 2020)

  Changed paths:
    M libcxx/utils/libcxx/test/config.py

  Log Message:
  -----------
  [libc++] NFC: Minor cleanups in config.py

Remove mentions of the ValgrindExecutor, which doesn't exist. That
executor is literally nowhere in the code base, so this is dead code
as far as we're concerned.

Also, inline a one-liner function that was called exactly once.


  Commit: 80aea8e4a8d1c8ea2c35e929858bde6273a42179
      https://github.com/llvm/llvm-project/commit/80aea8e4a8d1c8ea2c35e929858bde6273a42179
  Author: Louis Dionne <ldionne at apple.com>
  Date:   2020-04-14 (Tue, 14 Apr 2020)

  Changed paths:
    M libcxx/docs/DesignDocs/AvailabilityMarkup.rst
    M libcxx/test/libcxx/containers/sequences/array/array.zero/db_back.pass.cpp
    M libcxx/test/libcxx/containers/sequences/array/array.zero/db_front.pass.cpp
    M libcxx/test/libcxx/containers/sequences/array/array.zero/db_indexing.pass.cpp
    M libcxx/test/libcxx/containers/sequences/list/list.cons/db_copy.pass.cpp
    M libcxx/test/libcxx/containers/sequences/list/list.cons/db_move.pass.cpp
    M libcxx/test/libcxx/containers/sequences/list/list.modifiers/emplace_db1.pass.cpp
    M libcxx/test/libcxx/containers/sequences/list/list.modifiers/erase_iter_db1.pass.cpp
    M libcxx/test/libcxx/containers/sequences/list/list.modifiers/erase_iter_db2.pass.cpp
    M libcxx/test/libcxx/containers/sequences/list/list.modifiers/erase_iter_iter_db1.pass.cpp
    M libcxx/test/libcxx/containers/sequences/list/list.modifiers/erase_iter_iter_db2.pass.cpp
    M libcxx/test/libcxx/containers/sequences/list/list.modifiers/erase_iter_iter_db3.pass.cpp
    M libcxx/test/libcxx/containers/sequences/list/list.modifiers/erase_iter_iter_db4.pass.cpp
    M libcxx/test/libcxx/containers/sequences/list/list.modifiers/insert_iter_iter_iter_db1.pass.cpp
    M libcxx/test/libcxx/containers/sequences/list/list.modifiers/insert_iter_rvalue_db1.pass.cpp
    M libcxx/test/libcxx/containers/sequences/list/list.modifiers/insert_iter_size_value_db1.pass.cpp
    M libcxx/test/libcxx/containers/sequences/list/list.modifiers/insert_iter_value_db1.pass.cpp
    M libcxx/test/libcxx/containers/sequences/list/list.modifiers/pop_back_db1.pass.cpp
    M libcxx/test/libcxx/containers/sequences/list/list.ops/db_splice_pos_list.pass.cpp
    M libcxx/test/libcxx/containers/sequences/list/list.ops/db_splice_pos_list_iter.pass.cpp
    M libcxx/test/libcxx/containers/sequences/list/list.ops/db_splice_pos_list_iter_iter.pass.cpp
    M libcxx/test/libcxx/debug/containers/db_associative_container_tests.pass.cpp
    M libcxx/test/libcxx/debug/containers/db_sequence_container_iterators.multithread.pass.cpp
    M libcxx/test/libcxx/debug/containers/db_sequence_container_iterators.pass.cpp
    M libcxx/test/libcxx/debug/containers/db_string.pass.cpp
    M libcxx/test/libcxx/debug/containers/db_unord_container_tests.pass.cpp
    M libcxx/test/libcxx/debug/db_string_view.pass.cpp
    M libcxx/test/libcxx/debug/debug_abort.pass.cpp
    M libcxx/test/libcxx/debug/debug_helper_test.pass.cpp
    M libcxx/test/libcxx/debug/debug_register.pass.cpp
    M libcxx/test/libcxx/iterators/advance.debug1.pass.cpp
    M libcxx/test/libcxx/iterators/next.debug1.pass.cpp
    M libcxx/test/libcxx/iterators/prev.debug1.pass.cpp
    M libcxx/utils/libcxx/test/config.py

  Log Message:
  -----------
  [libc++] Simplify how with_system_cxx_lib and availability features are defined

Instead of spamming a bunch of available features that are not actually
used anywhere, only set those that are actually used in the test suite.
In the future, this should probably be based on the target triple only,
with the ability to have wildcards in the triple.


Compare: https://github.com/llvm/llvm-project/compare/426f37584ec3...80aea8e4a8d1


More information about the All-commits mailing list