[PATCH] D68081: Allow update_test_checks.py to not scrub names

Simon Tatham via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 8 08:26:05 PDT 2019


simon_tatham added a comment.

I had `update_cc_test_checks.py` give me a Python traceback just now which I think must be to do with this commit:

  Traceback (most recent call last):
    File "llvm/utils/update_cc_test_checks.py", line 267, in <module>
      sys.exit(main())
    File "llvm/utils/update_cc_test_checks.py", line 255, in main
      common.add_ir_checks(output_lines, '//', run_list, func_dict, mangled)
  TypeError: add_ir_checks() missing 1 required positional argument: 'preserve_names'

The new parameter to `add_ir_checks` is mandatory, but it's only been added to the call in `update_test_checks.py`, and not to the one in `update_cc_test_checks.py`.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D68081/new/

https://reviews.llvm.org/D68081





More information about the llvm-commits mailing list