2018-02-03 00:41:07 +08:00
|
|
|
from __future__ import print_function
|
2020-07-07 23:38:28 +08:00
|
|
|
|
|
|
|
import copy
|
|
|
|
import glob
|
2018-01-30 08:40:05 +08:00
|
|
|
import re
|
|
|
|
import subprocess
|
2018-02-03 00:41:07 +08:00
|
|
|
import sys
|
2018-01-30 08:40:05 +08:00
|
|
|
|
2018-02-10 13:01:33 +08:00
|
|
|
if sys.version_info[0] > 2:
|
|
|
|
class string:
|
|
|
|
expandtabs = str.expandtabs
|
|
|
|
else:
|
|
|
|
import string
|
2018-01-30 08:40:05 +08:00
|
|
|
|
2018-02-10 13:01:33 +08:00
|
|
|
##### Common utilities for update_*test_checks.py
|
2018-01-30 08:40:05 +08:00
|
|
|
|
2019-11-20 21:19:48 +08:00
|
|
|
|
2019-12-02 18:50:23 +08:00
|
|
|
_verbose = False
|
|
|
|
|
2019-11-20 21:19:48 +08:00
|
|
|
def parse_commandline_args(parser):
|
|
|
|
parser.add_argument('-v', '--verbose', action='store_true',
|
|
|
|
help='Show verbose output')
|
|
|
|
parser.add_argument('-u', '--update-only', action='store_true',
|
|
|
|
help='Only update test if it was already autogened')
|
2020-07-07 23:38:28 +08:00
|
|
|
parser.add_argument('--force-update', action='store_true',
|
|
|
|
help='Update test even if it was autogened by a different script')
|
|
|
|
parser.add_argument('--enable', action='store_true', dest='enabled', default=True,
|
|
|
|
help='Activate CHECK line generation from this point forward')
|
|
|
|
parser.add_argument('--disable', action='store_false', dest='enabled',
|
|
|
|
help='Deactivate CHECK line generation from this point forward')
|
2019-12-02 18:50:23 +08:00
|
|
|
args = parser.parse_args()
|
|
|
|
global _verbose
|
|
|
|
_verbose = args.verbose
|
|
|
|
return args
|
2019-11-20 21:19:48 +08:00
|
|
|
|
2020-07-07 23:38:28 +08:00
|
|
|
|
|
|
|
class InputLineInfo(object):
|
|
|
|
def __init__(self, line, line_number, args, argv):
|
|
|
|
self.line = line
|
|
|
|
self.line_number = line_number
|
|
|
|
self.args = args
|
|
|
|
self.argv = argv
|
|
|
|
|
|
|
|
|
|
|
|
class TestInfo(object):
|
|
|
|
def __init__(self, test, parser, script_name, input_lines, args, argv,
|
2020-08-03 18:18:01 +08:00
|
|
|
comment_prefix, argparse_callback):
|
2020-07-07 23:38:28 +08:00
|
|
|
self.parser = parser
|
2020-08-03 18:18:01 +08:00
|
|
|
self.argparse_callback = argparse_callback
|
2020-07-07 23:38:28 +08:00
|
|
|
self.path = test
|
|
|
|
self.args = args
|
|
|
|
self.argv = argv
|
|
|
|
self.input_lines = input_lines
|
|
|
|
self.run_lines = find_run_lines(test, self.input_lines)
|
|
|
|
self.comment_prefix = comment_prefix
|
|
|
|
if self.comment_prefix is None:
|
|
|
|
if self.path.endswith('.mir'):
|
|
|
|
self.comment_prefix = '#'
|
|
|
|
else:
|
|
|
|
self.comment_prefix = ';'
|
|
|
|
self.autogenerated_note_prefix = self.comment_prefix + ' ' + UTC_ADVERT
|
|
|
|
self.test_autogenerated_note = self.autogenerated_note_prefix + script_name
|
|
|
|
self.test_autogenerated_note += get_autogennote_suffix(parser, self.args)
|
|
|
|
|
|
|
|
def iterlines(self, output_lines):
|
|
|
|
output_lines.append(self.test_autogenerated_note)
|
|
|
|
for line_num, input_line in enumerate(self.input_lines):
|
|
|
|
# Discard any previous script advertising.
|
|
|
|
if input_line.startswith(self.autogenerated_note_prefix):
|
|
|
|
continue
|
|
|
|
self.args, self.argv = check_for_command(input_line, self.parser,
|
2020-08-03 18:18:01 +08:00
|
|
|
self.args, self.argv, self.argparse_callback)
|
2020-07-07 23:38:28 +08:00
|
|
|
if not self.args.enabled:
|
|
|
|
output_lines.append(input_line)
|
|
|
|
continue
|
|
|
|
yield InputLineInfo(input_line, line_num, self.args, self.argv)
|
|
|
|
|
|
|
|
|
2020-08-03 18:18:01 +08:00
|
|
|
def itertests(test_patterns, parser, script_name, comment_prefix=None, argparse_callback=None):
|
2020-07-07 23:38:28 +08:00
|
|
|
for pattern in test_patterns:
|
|
|
|
# On Windows we must expand the patterns ourselves.
|
|
|
|
tests_list = glob.glob(pattern)
|
|
|
|
if not tests_list:
|
|
|
|
warn("Test file pattern '%s' was not found. Ignoring it." % (pattern,))
|
|
|
|
continue
|
|
|
|
for test in tests_list:
|
|
|
|
with open(test) as f:
|
|
|
|
input_lines = [l.rstrip() for l in f]
|
|
|
|
args = parser.parse_args()
|
2020-08-03 18:18:01 +08:00
|
|
|
if argparse_callback is not None:
|
|
|
|
argparse_callback(args)
|
2020-07-07 23:38:28 +08:00
|
|
|
argv = sys.argv[:]
|
|
|
|
first_line = input_lines[0] if input_lines else ""
|
|
|
|
if UTC_ADVERT in first_line:
|
|
|
|
if script_name not in first_line and not args.force_update:
|
|
|
|
warn("Skipping test which wasn't autogenerated by " + script_name, test)
|
|
|
|
continue
|
2020-08-03 18:18:01 +08:00
|
|
|
args, argv = check_for_command(first_line, parser, args, argv, argparse_callback)
|
2020-07-07 23:38:28 +08:00
|
|
|
elif args.update_only:
|
|
|
|
assert UTC_ADVERT not in first_line
|
|
|
|
warn("Skipping test which isn't autogenerated: " + test)
|
|
|
|
continue
|
|
|
|
yield TestInfo(test, parser, script_name, input_lines, args, argv,
|
2020-08-03 18:18:01 +08:00
|
|
|
comment_prefix, argparse_callback)
|
2020-07-07 23:38:28 +08:00
|
|
|
|
|
|
|
|
2018-01-30 08:40:05 +08:00
|
|
|
def should_add_line_to_output(input_line, prefix_set):
|
|
|
|
# Skip any blank comment lines in the IR.
|
|
|
|
if input_line.strip() == ';':
|
|
|
|
return False
|
|
|
|
# Skip any blank lines in the IR.
|
|
|
|
#if input_line.strip() == '':
|
|
|
|
# return False
|
|
|
|
# And skip any CHECK lines. We're building our own.
|
|
|
|
m = CHECK_RE.match(input_line)
|
|
|
|
if m and m.group(1) in prefix_set:
|
|
|
|
return False
|
|
|
|
|
|
|
|
return True
|
|
|
|
|
|
|
|
# Invoke the tool that is being tested.
|
|
|
|
def invoke_tool(exe, cmd_args, ir):
|
|
|
|
with open(ir) as ir_file:
|
[utils] Add utils/update_cc_test_checks.py
A utility to update LLVM IR in C/C++ FileCheck test files.
Example RUN lines in .c/.cc test files:
// RUN: %clang -S -Os -DXX %s -o - | FileCheck %s
// RUN: %clangxx -S -Os %s -o - | FileCheck -check-prefix=IR %s
Usage:
% utils/update_cc_test_checks.py --llvm-bin=release/bin test/a.cc
% utils/update_cc_test_checks.py --c-index-test=release/bin/c-index-test --clang=release/bin/clang /tmp/c/a.cc
// NOTE: Assertions have been autogenerated by utils/update_cc_test_checks.py
// RUN: %clang -emit-llvm -S -Os -DXX %s -o - | FileCheck -check-prefix=AA %s
// RUN: %clangxx -emit-llvm -S -Os %s -o - | FileCheck -check-prefix=BB %s
using T =
#ifdef XX
int __attribute__((vector_size(16)))
#else
short __attribute__((vector_size(16)))
#endif
;
// AA-LABEL: _Z3fooDv4_i:
// AA: entry:
// AA-NEXT: %add = shl <4 x i32> %a, <i32 1, i32 1, i32 1, i32 1>
// AA-NEXT: ret <4 x i32> %add
//
// BB-LABEL: _Z3fooDv8_s:
// BB: entry:
// BB-NEXT: %add = shl <8 x i16> %a, <i16 1, i16 1, i16 1, i16 1, i16 1, i16 1, i16 1, i16 1>
// BB-NEXT: ret <8 x i16> %add
T foo(T a) {
return a + a;
}
Differential Revision: https://reviews.llvm.org/D42712
llvm-svn: 326591
2018-03-03 01:37:04 +08:00
|
|
|
# TODO Remove the str form which is used by update_test_checks.py and
|
|
|
|
# update_llc_test_checks.py
|
|
|
|
# The safer list form is used by update_cc_test_checks.py
|
|
|
|
if isinstance(cmd_args, list):
|
|
|
|
stdout = subprocess.check_output([exe] + cmd_args, stdin=ir_file)
|
|
|
|
else:
|
|
|
|
stdout = subprocess.check_output(exe + ' ' + cmd_args,
|
|
|
|
shell=True, stdin=ir_file)
|
2018-02-03 00:41:07 +08:00
|
|
|
if sys.version_info[0] > 2:
|
|
|
|
stdout = stdout.decode()
|
2018-01-30 08:40:05 +08:00
|
|
|
# Fix line endings to unix CR style.
|
2018-02-03 00:41:07 +08:00
|
|
|
return stdout.replace('\r\n', '\n')
|
2018-01-30 08:40:05 +08:00
|
|
|
|
2018-02-10 13:01:33 +08:00
|
|
|
##### LLVM IR parser
|
2019-12-02 18:50:23 +08:00
|
|
|
RUN_LINE_RE = re.compile(r'^\s*(?://|[;#])\s*RUN:\s*(.*)$')
|
2019-10-30 17:35:10 +08:00
|
|
|
CHECK_PREFIX_RE = re.compile(r'--?check-prefix(?:es)?[= ](\S+)')
|
[UpdateTestChecks] Emit warning when invalid value for -check-prefix(es) option
Summary:
The script is silent for the following issue:
FileCheck %s -check-prefix=CHECK,POPCOUNT
FileCheck will catch it later, but I think we can warn here too.
Now it warns:
./update_llc_test_checks.py file.ll
WARNING: Supplied prefix 'CHECK,POPCOUNT' is invalid. Prefix must contain only alphanumeric characters, hyphens and underscores. Did you mean --check-prefixes=CHECK,POPCOUNT?
Reviewers: lebedev.ri, spatel, RKSimon, craig.topper, nikic, gbedwell
Reviewed By: RKSimon
Subscribers: llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D64589
llvm-svn: 367244
2019-07-30 01:41:00 +08:00
|
|
|
PREFIX_RE = re.compile('^[a-zA-Z0-9_-]+$')
|
2019-12-22 06:18:38 +08:00
|
|
|
CHECK_RE = re.compile(r'^\s*(?://|[;#])\s*([^:]+?)(?:-NEXT|-NOT|-DAG|-LABEL|-SAME|-EMPTY)?:')
|
2018-02-10 13:01:33 +08:00
|
|
|
|
2019-11-01 11:45:17 +08:00
|
|
|
UTC_ARGS_KEY = 'UTC_ARGS:'
|
|
|
|
UTC_ARGS_CMD = re.compile(r'.*' + UTC_ARGS_KEY + '\s*(?P<cmd>.*)\s*$')
|
2020-07-07 23:38:28 +08:00
|
|
|
UTC_ADVERT = 'NOTE: Assertions have been autogenerated by '
|
2019-11-01 11:45:17 +08:00
|
|
|
|
2018-02-10 13:01:33 +08:00
|
|
|
OPT_FUNCTION_RE = re.compile(
|
2020-01-24 04:30:32 +08:00
|
|
|
r'^(\s*;\s*Function\sAttrs:\s(?P<attrs>[\w\s]+?))?\s*define\s+(?:internal\s+)?[^@]*@(?P<func>[\w.$-]+?)\s*'
|
2020-08-28 16:19:04 +08:00
|
|
|
r'(?P<args_and_sig>\((\)|(.*?[\w.-]+?)\))[^{]*\{)\n(?P<body>.*?)^\}$',
|
2018-02-10 13:01:33 +08:00
|
|
|
flags=(re.M | re.S))
|
|
|
|
|
2018-04-06 20:36:27 +08:00
|
|
|
ANALYZE_FUNCTION_RE = re.compile(
|
2020-01-24 04:30:32 +08:00
|
|
|
r'^\s*\'(?P<analysis>[\w\s-]+?)\'\s+for\s+function\s+\'(?P<func>[\w.$-]+?)\':'
|
2018-04-06 20:36:27 +08:00
|
|
|
r'\s*\n(?P<body>.*)$',
|
|
|
|
flags=(re.X | re.S))
|
|
|
|
|
2020-01-24 04:30:32 +08:00
|
|
|
IR_FUNCTION_RE = re.compile(r'^\s*define\s+(?:internal\s+)?[^@]*@"?([\w.$-]+)"?\s*\(')
|
2018-02-28 08:56:24 +08:00
|
|
|
TRIPLE_IR_RE = re.compile(r'^\s*target\s+triple\s*=\s*"([^"]+)"$')
|
|
|
|
TRIPLE_ARG_RE = re.compile(r'-mtriple[= ]([^ ]+)')
|
|
|
|
MARCH_ARG_RE = re.compile(r'-march[= ]([^ ]+)')
|
2018-02-10 13:01:33 +08:00
|
|
|
|
|
|
|
SCRUB_LEADING_WHITESPACE_RE = re.compile(r'^(\s+)')
|
|
|
|
SCRUB_WHITESPACE_RE = re.compile(r'(?!^(| \w))[ \t]+', flags=re.M)
|
|
|
|
SCRUB_TRAILING_WHITESPACE_RE = re.compile(r'[ \t]+$', flags=re.M)
|
2019-10-11 09:32:04 +08:00
|
|
|
SCRUB_TRAILING_WHITESPACE_TEST_RE = SCRUB_TRAILING_WHITESPACE_RE
|
|
|
|
SCRUB_TRAILING_WHITESPACE_AND_ATTRIBUTES_RE = re.compile(r'([ \t]|(#[0-9]+))+$', flags=re.M)
|
2018-02-10 13:01:33 +08:00
|
|
|
SCRUB_KILL_COMMENT_RE = re.compile(r'^ *#+ +kill:.*\n')
|
|
|
|
SCRUB_LOOP_COMMENT_RE = re.compile(
|
|
|
|
r'# =>This Inner Loop Header:.*|# in Loop:.*', flags=re.M)
|
2020-04-10 14:09:01 +08:00
|
|
|
SCRUB_TAILING_COMMENT_TOKEN_RE = re.compile(r'(?<=\S)+[ \t]*#$', flags=re.M)
|
2018-02-10 13:01:33 +08:00
|
|
|
|
2019-08-07 22:44:50 +08:00
|
|
|
|
|
|
|
def error(msg, test_file=None):
|
|
|
|
if test_file:
|
|
|
|
msg = '{}: {}'.format(msg, test_file)
|
|
|
|
print('ERROR: {}'.format(msg), file=sys.stderr)
|
|
|
|
|
|
|
|
def warn(msg, test_file=None):
|
|
|
|
if test_file:
|
|
|
|
msg = '{}: {}'.format(msg, test_file)
|
|
|
|
print('WARNING: {}'.format(msg), file=sys.stderr)
|
|
|
|
|
2019-12-02 18:50:23 +08:00
|
|
|
def debug(*args, **kwargs):
|
|
|
|
# Python2 does not allow def debug(*args, file=sys.stderr, **kwargs):
|
|
|
|
if 'file' not in kwargs:
|
|
|
|
kwargs['file'] = sys.stderr
|
|
|
|
if _verbose:
|
|
|
|
print(*args, **kwargs)
|
|
|
|
|
|
|
|
def find_run_lines(test, lines):
|
|
|
|
debug('Scanning for RUN lines in test file:', test)
|
|
|
|
raw_lines = [m.group(1)
|
|
|
|
for m in [RUN_LINE_RE.match(l) for l in lines] if m]
|
|
|
|
run_lines = [raw_lines[0]] if len(raw_lines) > 0 else []
|
|
|
|
for l in raw_lines[1:]:
|
|
|
|
if run_lines[-1].endswith('\\'):
|
2019-12-03 16:23:25 +08:00
|
|
|
run_lines[-1] = run_lines[-1].rstrip('\\') + ' ' + l
|
2019-12-02 18:50:23 +08:00
|
|
|
else:
|
|
|
|
run_lines.append(l)
|
|
|
|
debug('Found {} RUN lines in {}:'.format(len(run_lines), test))
|
|
|
|
for l in run_lines:
|
|
|
|
debug(' RUN: {}'.format(l))
|
|
|
|
return run_lines
|
|
|
|
|
2018-02-10 13:01:33 +08:00
|
|
|
def scrub_body(body):
|
|
|
|
# Scrub runs of whitespace out of the assembly, but leave the leading
|
|
|
|
# whitespace in place.
|
|
|
|
body = SCRUB_WHITESPACE_RE.sub(r' ', body)
|
|
|
|
# Expand the tabs used for indentation.
|
|
|
|
body = string.expandtabs(body, 2)
|
|
|
|
# Strip trailing whitespace.
|
2019-10-11 09:32:04 +08:00
|
|
|
body = SCRUB_TRAILING_WHITESPACE_TEST_RE.sub(r'', body)
|
2018-02-10 13:01:33 +08:00
|
|
|
return body
|
|
|
|
|
2018-06-01 21:37:01 +08:00
|
|
|
def do_scrub(body, scrubber, scrubber_args, extra):
|
|
|
|
if scrubber_args:
|
|
|
|
local_args = copy.deepcopy(scrubber_args)
|
|
|
|
local_args[0].extra_scrub = extra
|
|
|
|
return scrubber(body, *local_args)
|
|
|
|
return scrubber(body, *scrubber_args)
|
|
|
|
|
2018-01-30 08:40:05 +08:00
|
|
|
# Build up a dictionary of all the function bodies.
|
2018-06-01 21:37:01 +08:00
|
|
|
class function_body(object):
|
2020-07-12 03:53:50 +08:00
|
|
|
def __init__(self, string, extra, args_and_sig, attrs):
|
2018-06-01 21:37:01 +08:00
|
|
|
self.scrub = string
|
|
|
|
self.extrascrub = extra
|
[Utils] Allow update_test_checks to check function information
Summary:
This adds a switch to the update_test_checks that triggers arguments and
other function annotations, e.g., personality, to be present in the
check line. If not set, the behavior should be the same as before.
If arguments are recorded, their names are scrubbed from the IR to allow
merging.
This patch includes D68153.
Reviewers: lebedev.ri, greened, spatel, xbolva00, RKSimon, mehdi_amini
Subscribers: bollu, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D68819
2019-10-11 01:08:21 +08:00
|
|
|
self.args_and_sig = args_and_sig
|
2020-07-12 03:53:50 +08:00
|
|
|
self.attrs = attrs
|
|
|
|
def is_same_except_arg_names(self, extrascrub, args_and_sig, attrs):
|
[Utils] Allow update_test_checks to check function information
Summary:
This adds a switch to the update_test_checks that triggers arguments and
other function annotations, e.g., personality, to be present in the
check line. If not set, the behavior should be the same as before.
If arguments are recorded, their names are scrubbed from the IR to allow
merging.
This patch includes D68153.
Reviewers: lebedev.ri, greened, spatel, xbolva00, RKSimon, mehdi_amini
Subscribers: bollu, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D68819
2019-10-11 01:08:21 +08:00
|
|
|
arg_names = set()
|
|
|
|
def drop_arg_names(match):
|
2020-08-11 02:59:07 +08:00
|
|
|
arg_names.add(match.group(3))
|
|
|
|
return match.group(1) + match.group(match.lastindex)
|
[Utils] Allow update_test_checks to check function information
Summary:
This adds a switch to the update_test_checks that triggers arguments and
other function annotations, e.g., personality, to be present in the
check line. If not set, the behavior should be the same as before.
If arguments are recorded, their names are scrubbed from the IR to allow
merging.
This patch includes D68153.
Reviewers: lebedev.ri, greened, spatel, xbolva00, RKSimon, mehdi_amini
Subscribers: bollu, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D68819
2019-10-11 01:08:21 +08:00
|
|
|
def repl_arg_names(match):
|
2020-08-29 21:35:14 +08:00
|
|
|
if match.group(3) is not None and match.group(3) in arg_names:
|
2020-08-11 02:59:07 +08:00
|
|
|
return match.group(1) + match.group(match.lastindex)
|
|
|
|
return match.group(1) + match.group(2) + match.group(match.lastindex)
|
2020-07-12 03:53:50 +08:00
|
|
|
if self.attrs != attrs:
|
|
|
|
return False
|
[Utils] Allow update_test_checks to check function information
Summary:
This adds a switch to the update_test_checks that triggers arguments and
other function annotations, e.g., personality, to be present in the
check line. If not set, the behavior should be the same as before.
If arguments are recorded, their names are scrubbed from the IR to allow
merging.
This patch includes D68153.
Reviewers: lebedev.ri, greened, spatel, xbolva00, RKSimon, mehdi_amini
Subscribers: bollu, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D68819
2019-10-11 01:08:21 +08:00
|
|
|
ans0 = IR_VALUE_RE.sub(drop_arg_names, self.args_and_sig)
|
|
|
|
ans1 = IR_VALUE_RE.sub(drop_arg_names, args_and_sig)
|
|
|
|
if ans0 != ans1:
|
|
|
|
return False
|
|
|
|
es0 = IR_VALUE_RE.sub(repl_arg_names, self.extrascrub)
|
|
|
|
es1 = IR_VALUE_RE.sub(repl_arg_names, extrascrub)
|
|
|
|
es0 = SCRUB_IR_COMMENT_RE.sub(r'', es0)
|
|
|
|
es1 = SCRUB_IR_COMMENT_RE.sub(r'', es1)
|
|
|
|
return es0 == es1
|
|
|
|
|
2018-06-01 21:37:01 +08:00
|
|
|
def __str__(self):
|
|
|
|
return self.scrub
|
|
|
|
|
2020-07-12 03:53:50 +08:00
|
|
|
def build_function_body_dictionary(function_re, scrubber, scrubber_args, raw_tool_output, prefixes, func_dict, verbose, record_args, check_attributes):
|
2018-01-30 08:40:05 +08:00
|
|
|
for m in function_re.finditer(raw_tool_output):
|
|
|
|
if not m:
|
|
|
|
continue
|
|
|
|
func = m.group('func')
|
2018-06-01 21:37:01 +08:00
|
|
|
body = m.group('body')
|
2020-07-12 03:53:50 +08:00
|
|
|
attrs = m.group('attrs') if check_attributes else ''
|
[Utils] Allow update_test_checks to check function information
Summary:
This adds a switch to the update_test_checks that triggers arguments and
other function annotations, e.g., personality, to be present in the
check line. If not set, the behavior should be the same as before.
If arguments are recorded, their names are scrubbed from the IR to allow
merging.
This patch includes D68153.
Reviewers: lebedev.ri, greened, spatel, xbolva00, RKSimon, mehdi_amini
Subscribers: bollu, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D68819
2019-10-11 01:08:21 +08:00
|
|
|
# Determine if we print arguments, the opening brace, or nothing after the function name
|
|
|
|
if record_args and 'args_and_sig' in m.groupdict():
|
|
|
|
args_and_sig = scrub_body(m.group('args_and_sig').strip())
|
|
|
|
elif 'args_and_sig' in m.groupdict():
|
|
|
|
args_and_sig = '('
|
|
|
|
else:
|
|
|
|
args_and_sig = ''
|
2018-06-01 21:37:01 +08:00
|
|
|
scrubbed_body = do_scrub(body, scrubber, scrubber_args, extra = False)
|
|
|
|
scrubbed_extra = do_scrub(body, scrubber, scrubber_args, extra = True)
|
2018-12-07 17:49:21 +08:00
|
|
|
if 'analysis' in m.groupdict():
|
2018-04-06 20:36:27 +08:00
|
|
|
analysis = m.group('analysis')
|
|
|
|
if analysis.lower() != 'cost model analysis':
|
2019-08-07 22:44:50 +08:00
|
|
|
warn('Unsupported analysis mode: %r!' % (analysis,))
|
2018-01-30 08:40:05 +08:00
|
|
|
if func.startswith('stress'):
|
|
|
|
# We only use the last line of the function body for stress tests.
|
|
|
|
scrubbed_body = '\n'.join(scrubbed_body.splitlines()[-1:])
|
|
|
|
if verbose:
|
2018-02-03 00:41:07 +08:00
|
|
|
print('Processing function: ' + func, file=sys.stderr)
|
2018-01-30 08:40:05 +08:00
|
|
|
for l in scrubbed_body.splitlines():
|
2018-02-03 00:41:07 +08:00
|
|
|
print(' ' + l, file=sys.stderr)
|
2018-01-30 08:40:05 +08:00
|
|
|
for prefix in prefixes:
|
2020-07-12 03:53:50 +08:00
|
|
|
if func in func_dict[prefix]:
|
|
|
|
if str(func_dict[prefix][func]) != scrubbed_body or (func_dict[prefix][func] and (func_dict[prefix][func].args_and_sig != args_and_sig or func_dict[prefix][func].attrs != attrs)):
|
|
|
|
if func_dict[prefix][func] and func_dict[prefix][func].is_same_except_arg_names(scrubbed_extra, args_and_sig, attrs):
|
|
|
|
func_dict[prefix][func].scrub = scrubbed_extra
|
|
|
|
func_dict[prefix][func].args_and_sig = args_and_sig
|
2018-06-01 21:37:01 +08:00
|
|
|
continue
|
2020-07-12 03:53:50 +08:00
|
|
|
else:
|
|
|
|
if prefix == prefixes[-1]:
|
|
|
|
warn('Found conflicting asm under the same prefix: %r!' % (prefix,))
|
|
|
|
else:
|
|
|
|
func_dict[prefix][func] = None
|
|
|
|
continue
|
2018-01-30 08:40:05 +08:00
|
|
|
|
2020-07-12 03:53:50 +08:00
|
|
|
func_dict[prefix][func] = function_body(scrubbed_body, scrubbed_extra, args_and_sig, attrs)
|
2018-02-10 13:01:33 +08:00
|
|
|
|
|
|
|
##### Generator of LLVM IR CHECK lines
|
|
|
|
|
|
|
|
SCRUB_IR_COMMENT_RE = re.compile(r'\s*;.*')
|
|
|
|
|
2020-08-11 02:59:07 +08:00
|
|
|
# TODO: We should also derive check lines for global, debug, loop declarations, etc..
|
|
|
|
|
|
|
|
class NamelessValue:
|
|
|
|
def __init__(self, check_prefix, ir_prefix, ir_regexp):
|
|
|
|
self.check_prefix = check_prefix
|
|
|
|
self.ir_prefix = ir_prefix
|
|
|
|
self.ir_regexp = ir_regexp
|
|
|
|
|
|
|
|
# Description of the different "unnamed" values we match in the IR, e.g.,
|
|
|
|
# (local) ssa values, (debug) metadata, etc.
|
|
|
|
nameless_values = [
|
|
|
|
NamelessValue(r'TMP', r'%', r'[\w.-]+?'),
|
|
|
|
NamelessValue(r'GLOB', r'@', r'[0-9]+?'),
|
|
|
|
NamelessValue(r'ATTR', r'#', r'[0-9]+?'),
|
|
|
|
NamelessValue(r'DBG', r'!dbg !', r'[0-9]+?'),
|
|
|
|
NamelessValue(r'TBAA', r'!tbaa !', r'[0-9]+?'),
|
|
|
|
NamelessValue(r'RNG', r'!range !', r'[0-9]+?'),
|
|
|
|
NamelessValue(r'LOOP', r'!llvm.loop !', r'[0-9]+?'),
|
|
|
|
NamelessValue(r'META', r'metadata !', r'[0-9]+?'),
|
|
|
|
]
|
|
|
|
|
|
|
|
# Build the regexp that matches an "IR value". This can be a local variable,
|
|
|
|
# argument, global, or metadata, anything that is "named". It is important that
|
|
|
|
# the PREFIX and SUFFIX below only contain a single group, if that changes
|
|
|
|
# other locations will need adjustment as well.
|
|
|
|
IR_VALUE_REGEXP_PREFIX = r'(\s+)'
|
|
|
|
IR_VALUE_REGEXP_STRING = r''
|
|
|
|
for nameless_value in nameless_values:
|
|
|
|
if IR_VALUE_REGEXP_STRING:
|
|
|
|
IR_VALUE_REGEXP_STRING += '|'
|
|
|
|
IR_VALUE_REGEXP_STRING += nameless_value.ir_prefix + r'(' + nameless_value.ir_regexp + r')'
|
|
|
|
IR_VALUE_REGEXP_SUFFIX = r'([,\s\(\)]|\Z)'
|
|
|
|
IR_VALUE_RE = re.compile(IR_VALUE_REGEXP_PREFIX + r'(' + IR_VALUE_REGEXP_STRING + r')' + IR_VALUE_REGEXP_SUFFIX)
|
|
|
|
|
|
|
|
# The entire match is group 0, the prefix has one group (=1), the entire
|
|
|
|
# IR_VALUE_REGEXP_STRING is one group (=2), and then the nameless values start.
|
|
|
|
first_nameless_group_in_ir_value_match = 3
|
|
|
|
|
|
|
|
# Check a match for IR_VALUE_RE and inspect it to determine if it was a local
|
|
|
|
# value, %..., global @..., debug number !dbg !..., etc. See the PREFIXES above.
|
|
|
|
def get_idx_from_ir_value_match(match):
|
|
|
|
for i in range(first_nameless_group_in_ir_value_match, match.lastindex):
|
|
|
|
if match.group(i) is not None:
|
|
|
|
return i - first_nameless_group_in_ir_value_match
|
|
|
|
error("Unable to identify the kind of IR value from the match!")
|
|
|
|
return 0;
|
|
|
|
|
|
|
|
# See get_idx_from_ir_value_match
|
|
|
|
def get_name_from_ir_value_match(match):
|
|
|
|
return match.group(get_idx_from_ir_value_match(match) + first_nameless_group_in_ir_value_match)
|
|
|
|
|
|
|
|
# Return the nameless prefix we use for this kind or IR value, see also
|
|
|
|
# get_idx_from_ir_value_match
|
|
|
|
def get_nameless_check_prefix_from_ir_value_match(match):
|
|
|
|
return nameless_values[get_idx_from_ir_value_match(match)].check_prefix
|
|
|
|
|
|
|
|
# Return the IR prefix we use for this kind or IR value, e.g., % for locals,
|
|
|
|
# see also get_idx_from_ir_value_match
|
|
|
|
def get_ir_prefix_from_ir_value_match(match):
|
|
|
|
return nameless_values[get_idx_from_ir_value_match(match)].ir_prefix
|
|
|
|
|
|
|
|
# Return true if this kind or IR value is "local", basically if it matches '%{{.*}}'.
|
|
|
|
def is_local_ir_value_match(match):
|
|
|
|
return nameless_values[get_idx_from_ir_value_match(match)].ir_prefix == '%'
|
2020-05-31 22:46:11 +08:00
|
|
|
|
2018-02-10 13:01:33 +08:00
|
|
|
# Create a FileCheck variable name based on an IR name.
|
2020-08-11 02:59:07 +08:00
|
|
|
def get_value_name(var, match):
|
2018-02-10 13:01:33 +08:00
|
|
|
if var.isdigit():
|
2020-08-11 02:59:07 +08:00
|
|
|
var = get_nameless_check_prefix_from_ir_value_match(match) + var
|
2018-02-10 13:01:33 +08:00
|
|
|
var = var.replace('.', '_')
|
2018-03-15 04:28:53 +08:00
|
|
|
var = var.replace('-', '_')
|
2018-02-10 13:01:33 +08:00
|
|
|
return var.upper()
|
|
|
|
|
|
|
|
# Create a FileCheck variable from regex.
|
2020-08-11 02:59:07 +08:00
|
|
|
def get_value_definition(var, match):
|
|
|
|
return '[[' + get_value_name(var, match) + ':' + get_ir_prefix_from_ir_value_match(match) + '.*]]'
|
2018-02-10 13:01:33 +08:00
|
|
|
|
|
|
|
# Use a FileCheck variable.
|
2020-08-11 02:59:07 +08:00
|
|
|
def get_value_use(var, match):
|
|
|
|
return '[[' + get_value_name(var, match) + ']]'
|
2018-02-10 13:01:33 +08:00
|
|
|
|
|
|
|
# Replace IR value defs and uses with FileCheck variables.
|
2020-08-14 09:25:02 +08:00
|
|
|
def generalize_check_lines(lines, is_analyze, vars_seen, global_vars_seen):
|
2018-02-10 13:01:33 +08:00
|
|
|
# This gets called for each match that occurs in
|
|
|
|
# a line. We transform variables we haven't seen
|
|
|
|
# into defs, and variables we have seen into uses.
|
|
|
|
def transform_line_vars(match):
|
2020-08-11 02:59:07 +08:00
|
|
|
pre = get_ir_prefix_from_ir_value_match(match)
|
|
|
|
var = get_name_from_ir_value_match(match)
|
|
|
|
for nameless_value in nameless_values:
|
2020-08-13 00:56:55 +08:00
|
|
|
if re.match(r'^' + nameless_value.check_prefix + r'[0-9]+?$', var, re.IGNORECASE):
|
2020-08-11 02:59:07 +08:00
|
|
|
warn("Change IR value name '%s' to prevent possible conflict with scripted FileCheck name." % (var,))
|
|
|
|
if (pre, var) in vars_seen or (pre, var) in global_vars_seen:
|
|
|
|
rv = get_value_use(var, match)
|
2018-02-10 13:01:33 +08:00
|
|
|
else:
|
2020-08-11 02:59:07 +08:00
|
|
|
if is_local_ir_value_match(match):
|
|
|
|
vars_seen.add((pre, var))
|
|
|
|
else:
|
|
|
|
global_vars_seen.add((pre, var))
|
|
|
|
rv = get_value_definition(var, match)
|
2018-02-10 13:01:33 +08:00
|
|
|
# re.sub replaces the entire regex match
|
|
|
|
# with whatever you return, so we have
|
|
|
|
# to make sure to hand it back everything
|
|
|
|
# including the commas and spaces.
|
2020-08-11 02:59:07 +08:00
|
|
|
return match.group(1) + rv + match.group(match.lastindex)
|
2018-02-10 13:01:33 +08:00
|
|
|
|
|
|
|
lines_with_def = []
|
|
|
|
|
|
|
|
for i, line in enumerate(lines):
|
|
|
|
# An IR variable named '%.' matches the FileCheck regex string.
|
|
|
|
line = line.replace('%.', '%dot')
|
|
|
|
# Ignore any comments, since the check lines will too.
|
|
|
|
scrubbed_line = SCRUB_IR_COMMENT_RE.sub(r'', line)
|
2020-08-11 02:59:07 +08:00
|
|
|
lines[i] = scrubbed_line
|
|
|
|
if not is_analyze:
|
|
|
|
# It can happen that two matches are back-to-back and for some reason sub
|
|
|
|
# will not replace both of them. For now we work around this by
|
|
|
|
# substituting until there is no more match.
|
|
|
|
changed = True
|
|
|
|
while changed:
|
|
|
|
(lines[i], changed) = IR_VALUE_RE.subn(transform_line_vars, lines[i], count=1)
|
2018-02-10 13:01:33 +08:00
|
|
|
return lines
|
|
|
|
|
|
|
|
|
2020-08-11 02:59:07 +08:00
|
|
|
def add_checks(output_lines, comment_marker, prefix_list, func_dict, func_name, check_label_format, is_asm, is_analyze, global_vars_seen_dict):
|
2020-06-20 12:30:17 +08:00
|
|
|
# prefix_exclusions are prefixes we cannot use to print the function because it doesn't exist in run lines that use these prefixes as well.
|
|
|
|
prefix_exclusions = set()
|
2018-02-10 13:01:33 +08:00
|
|
|
printed_prefixes = []
|
2018-03-15 01:47:07 +08:00
|
|
|
for p in prefix_list:
|
|
|
|
checkprefixes = p[0]
|
2019-10-11 09:23:17 +08:00
|
|
|
# If not all checkprefixes of this run line produced the function we cannot check for it as it does not
|
|
|
|
# exist for this run line. A subset of the check prefixes might know about the function but only because
|
|
|
|
# other run lines created it.
|
|
|
|
if any(map(lambda checkprefix: func_name not in func_dict[checkprefix], checkprefixes)):
|
2020-06-20 12:30:17 +08:00
|
|
|
prefix_exclusions |= set(checkprefixes)
|
2019-10-11 09:23:17 +08:00
|
|
|
continue
|
|
|
|
|
2020-06-20 12:30:17 +08:00
|
|
|
# prefix_exclusions is constructed, we can now emit the output
|
2019-10-11 09:23:17 +08:00
|
|
|
for p in prefix_list:
|
|
|
|
checkprefixes = p[0]
|
2018-02-10 13:01:33 +08:00
|
|
|
for checkprefix in checkprefixes:
|
|
|
|
if checkprefix in printed_prefixes:
|
|
|
|
break
|
2019-10-11 09:23:17 +08:00
|
|
|
|
2020-06-20 12:30:17 +08:00
|
|
|
# Check if the prefix is excluded.
|
|
|
|
if checkprefix in prefix_exclusions:
|
2020-04-02 10:54:46 +08:00
|
|
|
continue
|
2019-10-11 09:23:17 +08:00
|
|
|
|
2020-04-02 10:54:46 +08:00
|
|
|
# If we do not have output for this prefix we skip it.
|
2019-10-11 09:23:17 +08:00
|
|
|
if not func_dict[checkprefix][func_name]:
|
2020-04-02 10:54:46 +08:00
|
|
|
continue
|
2018-04-05 18:26:13 +08:00
|
|
|
|
2018-02-10 13:01:33 +08:00
|
|
|
# Add some space between different check prefixes, but not after the last
|
|
|
|
# check line (before the test code).
|
2019-07-22 12:59:01 +08:00
|
|
|
if is_asm:
|
2018-04-05 18:48:38 +08:00
|
|
|
if len(printed_prefixes) != 0:
|
|
|
|
output_lines.append(comment_marker)
|
2018-04-05 18:26:13 +08:00
|
|
|
|
2020-08-11 02:59:07 +08:00
|
|
|
if checkprefix not in global_vars_seen_dict:
|
|
|
|
global_vars_seen_dict[checkprefix] = set()
|
|
|
|
global_vars_seen = global_vars_seen_dict[checkprefix]
|
|
|
|
|
2019-11-02 01:51:26 +08:00
|
|
|
vars_seen = set()
|
2018-02-10 13:01:33 +08:00
|
|
|
printed_prefixes.append(checkprefix)
|
2020-07-12 03:53:50 +08:00
|
|
|
attrs = str(func_dict[checkprefix][func_name].attrs)
|
|
|
|
attrs = '' if attrs == 'None' else attrs
|
|
|
|
if attrs:
|
2020-07-20 02:45:24 +08:00
|
|
|
output_lines.append('%s %s: Function Attrs: %s' % (comment_marker, checkprefix, attrs))
|
[Utils] Allow update_test_checks to check function information
Summary:
This adds a switch to the update_test_checks that triggers arguments and
other function annotations, e.g., personality, to be present in the
check line. If not set, the behavior should be the same as before.
If arguments are recorded, their names are scrubbed from the IR to allow
merging.
This patch includes D68153.
Reviewers: lebedev.ri, greened, spatel, xbolva00, RKSimon, mehdi_amini
Subscribers: bollu, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D68819
2019-10-11 01:08:21 +08:00
|
|
|
args_and_sig = str(func_dict[checkprefix][func_name].args_and_sig)
|
2020-08-14 09:25:02 +08:00
|
|
|
args_and_sig = generalize_check_lines([args_and_sig], is_analyze, vars_seen, global_vars_seen)[0]
|
[Utils] Allow update_test_checks to check function information
Summary:
This adds a switch to the update_test_checks that triggers arguments and
other function annotations, e.g., personality, to be present in the
check line. If not set, the behavior should be the same as before.
If arguments are recorded, their names are scrubbed from the IR to allow
merging.
This patch includes D68153.
Reviewers: lebedev.ri, greened, spatel, xbolva00, RKSimon, mehdi_amini
Subscribers: bollu, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D68819
2019-10-11 01:08:21 +08:00
|
|
|
if '[[' in args_and_sig:
|
|
|
|
output_lines.append(check_label_format % (checkprefix, func_name, ''))
|
|
|
|
output_lines.append('%s %s-SAME: %s' % (comment_marker, checkprefix, args_and_sig))
|
|
|
|
else:
|
|
|
|
output_lines.append(check_label_format % (checkprefix, func_name, args_and_sig))
|
2018-06-01 21:37:01 +08:00
|
|
|
func_body = str(func_dict[checkprefix][func_name]).splitlines()
|
2018-02-10 13:01:33 +08:00
|
|
|
|
2018-04-05 18:48:38 +08:00
|
|
|
# For ASM output, just emit the check lines.
|
2019-07-22 12:59:01 +08:00
|
|
|
if is_asm:
|
2018-04-05 18:48:38 +08:00
|
|
|
output_lines.append('%s %s: %s' % (comment_marker, checkprefix, func_body[0]))
|
|
|
|
for func_line in func_body[1:]:
|
2019-10-31 00:52:16 +08:00
|
|
|
if func_line.strip() == '':
|
|
|
|
output_lines.append('%s %s-EMPTY:' % (comment_marker, checkprefix))
|
|
|
|
else:
|
|
|
|
output_lines.append('%s %s-NEXT: %s' % (comment_marker, checkprefix, func_line))
|
2018-04-05 18:48:38 +08:00
|
|
|
break
|
|
|
|
|
2018-02-10 13:01:33 +08:00
|
|
|
# For IR output, change all defs to FileCheck variables, so we're immune
|
|
|
|
# to variable naming fashions.
|
2020-08-14 09:25:02 +08:00
|
|
|
func_body = generalize_check_lines(func_body, is_analyze, vars_seen, global_vars_seen)
|
2018-02-10 13:01:33 +08:00
|
|
|
|
|
|
|
# This could be selectively enabled with an optional invocation argument.
|
|
|
|
# Disabled for now: better to check everything. Be safe rather than sorry.
|
|
|
|
|
|
|
|
# Handle the first line of the function body as a special case because
|
|
|
|
# it's often just noise (a useless asm comment or entry label).
|
|
|
|
#if func_body[0].startswith("#") or func_body[0].startswith("entry:"):
|
|
|
|
# is_blank_line = True
|
|
|
|
#else:
|
2018-04-05 18:26:13 +08:00
|
|
|
# output_lines.append('%s %s: %s' % (comment_marker, checkprefix, func_body[0]))
|
2018-02-10 13:01:33 +08:00
|
|
|
# is_blank_line = False
|
|
|
|
|
|
|
|
is_blank_line = False
|
|
|
|
|
|
|
|
for func_line in func_body:
|
|
|
|
if func_line.strip() == '':
|
|
|
|
is_blank_line = True
|
|
|
|
continue
|
|
|
|
# Do not waste time checking IR comments.
|
|
|
|
func_line = SCRUB_IR_COMMENT_RE.sub(r'', func_line)
|
|
|
|
|
|
|
|
# Skip blank lines instead of checking them.
|
2019-07-22 12:59:01 +08:00
|
|
|
if is_blank_line:
|
2018-03-15 01:47:07 +08:00
|
|
|
output_lines.append('{} {}: {}'.format(
|
|
|
|
comment_marker, checkprefix, func_line))
|
2018-02-10 13:01:33 +08:00
|
|
|
else:
|
2018-03-15 01:47:07 +08:00
|
|
|
output_lines.append('{} {}-NEXT: {}'.format(
|
|
|
|
comment_marker, checkprefix, func_line))
|
2018-02-10 13:01:33 +08:00
|
|
|
is_blank_line = False
|
|
|
|
|
|
|
|
# Add space between different check prefixes and also before the first
|
|
|
|
# line of code in the test function.
|
2018-03-15 01:47:07 +08:00
|
|
|
output_lines.append(comment_marker)
|
2018-02-10 13:01:33 +08:00
|
|
|
break
|
2018-04-05 18:26:13 +08:00
|
|
|
|
2019-10-07 22:37:20 +08:00
|
|
|
def add_ir_checks(output_lines, comment_marker, prefix_list, func_dict,
|
2020-08-11 02:59:07 +08:00
|
|
|
func_name, preserve_names, function_sig, global_vars_seen_dict):
|
2018-04-05 18:26:13 +08:00
|
|
|
# Label format is based on IR string.
|
2019-11-02 00:17:27 +08:00
|
|
|
function_def_regex = 'define {{[^@]+}}' if function_sig else ''
|
[Utils] Allow update_test_checks to check function information
Summary:
This adds a switch to the update_test_checks that triggers arguments and
other function annotations, e.g., personality, to be present in the
check line. If not set, the behavior should be the same as before.
If arguments are recorded, their names are scrubbed from the IR to allow
merging.
This patch includes D68153.
Reviewers: lebedev.ri, greened, spatel, xbolva00, RKSimon, mehdi_amini
Subscribers: bollu, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D68819
2019-10-11 01:08:21 +08:00
|
|
|
check_label_format = '{} %s-LABEL: {}@%s%s'.format(comment_marker, function_def_regex)
|
2019-10-07 22:37:20 +08:00
|
|
|
add_checks(output_lines, comment_marker, prefix_list, func_dict, func_name,
|
2020-08-11 02:59:07 +08:00
|
|
|
check_label_format, False, preserve_names, global_vars_seen_dict)
|
2018-04-06 20:36:27 +08:00
|
|
|
|
|
|
|
def add_analyze_checks(output_lines, comment_marker, prefix_list, func_dict, func_name):
|
[Utils] Allow update_test_checks to check function information
Summary:
This adds a switch to the update_test_checks that triggers arguments and
other function annotations, e.g., personality, to be present in the
check line. If not set, the behavior should be the same as before.
If arguments are recorded, their names are scrubbed from the IR to allow
merging.
This patch includes D68153.
Reviewers: lebedev.ri, greened, spatel, xbolva00, RKSimon, mehdi_amini
Subscribers: bollu, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D68819
2019-10-11 01:08:21 +08:00
|
|
|
check_label_format = '{} %s-LABEL: \'%s%s\''.format(comment_marker)
|
2020-08-25 18:12:26 +08:00
|
|
|
global_vars_seen_dict = {}
|
2020-08-11 02:59:07 +08:00
|
|
|
add_checks(output_lines, comment_marker, prefix_list, func_dict, func_name,
|
|
|
|
check_label_format, False, True, global_vars_seen_dict)
|
[UpdateTestChecks] Emit warning when invalid value for -check-prefix(es) option
Summary:
The script is silent for the following issue:
FileCheck %s -check-prefix=CHECK,POPCOUNT
FileCheck will catch it later, but I think we can warn here too.
Now it warns:
./update_llc_test_checks.py file.ll
WARNING: Supplied prefix 'CHECK,POPCOUNT' is invalid. Prefix must contain only alphanumeric characters, hyphens and underscores. Did you mean --check-prefixes=CHECK,POPCOUNT?
Reviewers: lebedev.ri, spatel, RKSimon, craig.topper, nikic, gbedwell
Reviewed By: RKSimon
Subscribers: llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D64589
llvm-svn: 367244
2019-07-30 01:41:00 +08:00
|
|
|
|
|
|
|
|
|
|
|
def check_prefix(prefix):
|
|
|
|
if not PREFIX_RE.match(prefix):
|
|
|
|
hint = ""
|
|
|
|
if ',' in prefix:
|
|
|
|
hint = " Did you mean '--check-prefixes=" + prefix + "'?"
|
2019-08-07 22:44:50 +08:00
|
|
|
warn(("Supplied prefix '%s' is invalid. Prefix must contain only alphanumeric characters, hyphens and underscores." + hint) %
|
|
|
|
(prefix))
|
[UpdateTestChecks] Emit warning when invalid value for -check-prefix(es) option
Summary:
The script is silent for the following issue:
FileCheck %s -check-prefix=CHECK,POPCOUNT
FileCheck will catch it later, but I think we can warn here too.
Now it warns:
./update_llc_test_checks.py file.ll
WARNING: Supplied prefix 'CHECK,POPCOUNT' is invalid. Prefix must contain only alphanumeric characters, hyphens and underscores. Did you mean --check-prefixes=CHECK,POPCOUNT?
Reviewers: lebedev.ri, spatel, RKSimon, craig.topper, nikic, gbedwell
Reviewed By: RKSimon
Subscribers: llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D64589
llvm-svn: 367244
2019-07-30 01:41:00 +08:00
|
|
|
|
|
|
|
|
|
|
|
def verify_filecheck_prefixes(fc_cmd):
|
|
|
|
fc_cmd_parts = fc_cmd.split()
|
|
|
|
for part in fc_cmd_parts:
|
|
|
|
if "check-prefix=" in part:
|
|
|
|
prefix = part.split('=', 1)[1]
|
|
|
|
check_prefix(prefix)
|
|
|
|
elif "check-prefixes=" in part:
|
|
|
|
prefixes = part.split('=', 1)[1].split(',')
|
|
|
|
for prefix in prefixes:
|
|
|
|
check_prefix(prefix)
|
|
|
|
if prefixes.count(prefix) > 1:
|
2019-08-07 22:44:50 +08:00
|
|
|
warn("Supplied prefix '%s' is not unique in the prefix list." % (prefix,))
|
2019-11-01 11:45:17 +08:00
|
|
|
|
2020-04-23 20:02:12 +08:00
|
|
|
|
2019-11-01 11:45:17 +08:00
|
|
|
def get_autogennote_suffix(parser, args):
|
2020-04-23 20:02:12 +08:00
|
|
|
autogenerated_note_args = ''
|
|
|
|
for action in parser._actions:
|
|
|
|
if not hasattr(args, action.dest):
|
|
|
|
continue # Ignore options such as --help that aren't included in args
|
|
|
|
# Ignore parameters such as paths to the binary or the list of tests
|
|
|
|
if action.dest in ('tests', 'update_only', 'opt_binary', 'llc_binary',
|
|
|
|
'clang', 'opt', 'llvm_bin', 'verbose'):
|
|
|
|
continue
|
|
|
|
value = getattr(args, action.dest)
|
|
|
|
if action.const is not None: # action stores a constant (usually True/False)
|
|
|
|
# Skip actions with different constant values (this happens with boolean
|
|
|
|
# --foo/--no-foo options)
|
|
|
|
if value != action.const:
|
|
|
|
continue
|
|
|
|
if parser.get_default(action.dest) == value:
|
|
|
|
continue # Don't add default values
|
|
|
|
autogenerated_note_args += action.option_strings[0] + ' '
|
|
|
|
if action.const is None: # action takes a parameter
|
|
|
|
autogenerated_note_args += '%s ' % value
|
|
|
|
if autogenerated_note_args:
|
|
|
|
autogenerated_note_args = ' %s %s' % (UTC_ARGS_KEY, autogenerated_note_args[:-1])
|
|
|
|
return autogenerated_note_args
|
2019-11-01 11:45:17 +08:00
|
|
|
|
|
|
|
|
2020-08-03 18:18:01 +08:00
|
|
|
def check_for_command(line, parser, args, argv, argparse_callback):
|
2019-11-01 11:45:17 +08:00
|
|
|
cmd_m = UTC_ARGS_CMD.match(line)
|
|
|
|
if cmd_m:
|
|
|
|
cmd = cmd_m.group('cmd').strip().split(' ')
|
|
|
|
argv = argv + cmd
|
|
|
|
args = parser.parse_args(filter(lambda arg: arg not in args.tests, argv))
|
2020-08-03 18:18:01 +08:00
|
|
|
if argparse_callback is not None:
|
|
|
|
argparse_callback(args)
|
2019-11-01 11:45:17 +08:00
|
|
|
return args, argv
|