[all-commits] [llvm/llvm-project] 8a42bf: [lit] Move the recursiveExpansionLimit setting to ...

Louis Dionne via All-commits all-commits at lists.llvm.org
Mon Apr 6 10:58:09 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 8a42bf24ae99d9d207ec140dadadc7d992bd2fa1
      https://github.com/llvm/llvm-project/commit/8a42bf24ae99d9d207ec140dadadc7d992bd2fa1
  Author: Louis Dionne <ldionne at apple.com>
  Date:   2020-04-06 (Mon, 06 Apr 2020)

  Changed paths:
    M libcxx/test/lit.cfg
    M libcxx/utils/libcxx/test/format.py
    M libcxx/utils/libcxx/test/newformat.py
    M libcxxabi/test/lit.cfg
    M llvm/docs/CommandGuide/lit.rst
    M llvm/utils/lit/lit/LitConfig.py
    M llvm/utils/lit/lit/TestRunner.py
    M llvm/utils/lit/lit/TestingConfig.py
    M llvm/utils/lit/tests/Inputs/shtest-recursive-substitution/does-not-substitute-within-limit/lit.cfg
    M llvm/utils/lit/tests/Inputs/shtest-recursive-substitution/negative-integer/lit.cfg
    M llvm/utils/lit/tests/Inputs/shtest-recursive-substitution/not-an-integer/lit.cfg
    M llvm/utils/lit/tests/Inputs/shtest-recursive-substitution/set-to-none/lit.cfg
    M llvm/utils/lit/tests/Inputs/shtest-recursive-substitution/substitutes-within-limit/lit.cfg

  Log Message:
  -----------
  [lit] Move the recursiveExpansionLimit setting to TestingConfig

The LitConfig is shared across the whole test suite. However, since
enabling recursive expansion can be a breaking change for some test
suites, it's important to confine the setting to test suites that
enable it explicitly.

Note that other issues were raised with the way recursiveExpansionLimit
operates. However, this commit simply moves the setting to the right
place -- the mechanism by which it works can be improved independently.

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




More information about the All-commits mailing list