[all-commits] [llvm/llvm-project] 8c0e40: utils/update_mir_test_checks.py: support UTC_ARGS

Nicolai Hähnle via All-commits all-commits at lists.llvm.org
Thu Dec 8 12:56:19 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 8c0e401da78d16eeabcf022763b48832fe62b1ed
      https://github.com/llvm/llvm-project/commit/8c0e401da78d16eeabcf022763b48832fe62b1ed
  Author: Gaëtan Bossu <gaetan.bossu at amd.com>
  Date:   2022-12-08 (Thu, 08 Dec 2022)

  Changed paths:
    M llvm/test/tools/UpdateTestChecks/update_mir_test_checks/Inputs/print-stack.mir.expected
    M llvm/test/tools/UpdateTestChecks/update_mir_test_checks/print-stack.test
    M llvm/utils/update_mir_test_checks.py

  Log Message:
  -----------
  utils/update_mir_test_checks.py: support UTC_ARGS

As a reminder, UTC_ARGS is used by lit test cases to specify which
arguments need to be passed to update_XXXX_test_checks.py to be
auto-updated properly.

The support is achieved by relying on common.itertests, which is what
other test
updaters use to iterate over test files.

This commit also changes how the --llc-binary option is saved in args.
It used to be saved as "llc", but it is here changed to the standard
"llc_binary" to make use of an existing ignore mechanism for specific
arguments. Without that change, the option would not be ignored and
would appear in UTC_ARGS. This would be different from what e.g.
update_llc_test_checks does. As update_mir_test_checks.py now supports
UTC_ARGS, it became important to ensure the option is ignored.

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




More information about the All-commits mailing list