vitalybuka added a comment. LGTM as it fixes update_cc_test_checks.py
================ Comment at: llvm/utils/update_cc_test_checks.py:116-126 +def infer_dependent_args(args): + if args.clang is None: + if args.llvm_bin is None: + args.clang = 'clang' + else: + args.clang = os.path.join(args.llvm_bin, 'clang') + if args.opt is None: ---------------- I guess empty clang also make no sense, "not args.clang" is better here Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84511/new/ https://reviews.llvm.org/D84511 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits