2019-01-03 23:44:24 +08:00
|
|
|
#!/usr/bin/env python
|
2016-03-25 07:19:26 +08:00
|
|
|
|
2017-06-13 01:44:30 +08:00
|
|
|
"""A script to generate FileCheck statements for 'opt' regression tests.
|
2016-03-25 07:19:26 +08:00
|
|
|
|
2017-06-13 01:44:30 +08:00
|
|
|
This script is a utility to update LLVM opt test cases with new
|
2016-03-25 07:19:26 +08:00
|
|
|
FileCheck patterns. It can either update all of the tests in the file or
|
|
|
|
a single test function.
|
2016-04-06 02:00:47 +08:00
|
|
|
|
|
|
|
Example usage:
|
2017-06-13 01:44:30 +08:00
|
|
|
$ update_test_checks.py --opt=../bin/opt test/foo.ll
|
2016-04-06 02:00:47 +08:00
|
|
|
|
|
|
|
Workflow:
|
|
|
|
1. Make a compiler patch that requires updating some number of FileCheck lines
|
|
|
|
in regression test files.
|
|
|
|
2. Save the patch and revert it from your local work area.
|
|
|
|
3. Update the RUN-lines in the affected regression tests to look canonical.
|
|
|
|
Example: "; RUN: opt < %s -instcombine -S | FileCheck %s"
|
|
|
|
4. Refresh the FileCheck lines for either the entire file or select functions by
|
|
|
|
running this script.
|
|
|
|
5. Commit the fresh baseline of checks.
|
|
|
|
6. Apply your patch from step 1 and rebuild your local binaries.
|
|
|
|
7. Re-run this script on affected regression tests.
|
|
|
|
8. Check the diffs to ensure the script has done something reasonable.
|
|
|
|
9. Submit a patch including the regression test diffs for review.
|
|
|
|
|
|
|
|
A common pattern is to have the script insert complete checking of every
|
|
|
|
instruction. Then, edit it down to only check the relevant instructions.
|
|
|
|
The script is designed to make adding checks to a test case fast, it is *not*
|
|
|
|
designed to be authoratitive about what constitutes a good test!
|
2016-03-25 07:19:26 +08:00
|
|
|
"""
|
|
|
|
|
2019-01-03 22:11:33 +08:00
|
|
|
from __future__ import print_function
|
|
|
|
|
2016-03-25 07:19:26 +08:00
|
|
|
import argparse
|
2019-03-05 18:44:37 +08:00
|
|
|
import glob
|
2016-03-25 07:19:26 +08:00
|
|
|
import itertools
|
|
|
|
import os # Used to advertise this file's name ("autogenerated_note").
|
|
|
|
import string
|
|
|
|
import subprocess
|
|
|
|
import sys
|
|
|
|
import tempfile
|
|
|
|
import re
|
|
|
|
|
2018-01-30 08:40:05 +08:00
|
|
|
from UpdateTestChecks import common
|
|
|
|
|
2016-04-06 03:50:21 +08:00
|
|
|
ADVERT = '; NOTE: Assertions have been autogenerated by '
|
2016-03-25 07:19:26 +08:00
|
|
|
|
|
|
|
# RegEx: this is where the magic happens.
|
|
|
|
|
2016-03-26 01:00:12 +08:00
|
|
|
IR_FUNCTION_RE = re.compile('^\s*define\s+(?:internal\s+)?[^@]*@([\w-]+)\s*\(')
|
2016-03-25 07:19:26 +08:00
|
|
|
|
|
|
|
|
2018-02-10 13:01:33 +08:00
|
|
|
|
2016-03-25 07:19:26 +08:00
|
|
|
|
|
|
|
|
|
|
|
def main():
|
2016-04-06 02:00:47 +08:00
|
|
|
from argparse import RawTextHelpFormatter
|
|
|
|
parser = argparse.ArgumentParser(description=__doc__, formatter_class=RawTextHelpFormatter)
|
2016-03-25 07:19:26 +08:00
|
|
|
parser.add_argument('-v', '--verbose', action='store_true',
|
|
|
|
help='Show verbose output')
|
2017-06-13 01:44:30 +08:00
|
|
|
parser.add_argument('--opt-binary', default='opt',
|
|
|
|
help='The opt binary used to generate the test case')
|
2016-03-25 07:19:26 +08:00
|
|
|
parser.add_argument(
|
|
|
|
'--function', help='The function in the test file to update')
|
|
|
|
parser.add_argument('tests', nargs='+')
|
|
|
|
args = parser.parse_args()
|
|
|
|
|
2016-04-06 03:50:21 +08:00
|
|
|
autogenerated_note = (ADVERT + 'utils/' + os.path.basename(__file__))
|
2016-03-25 07:19:26 +08:00
|
|
|
|
2017-06-13 01:44:30 +08:00
|
|
|
opt_basename = os.path.basename(args.opt_binary)
|
2019-05-12 12:55:09 +08:00
|
|
|
if not re.match(r'^opt(-\d+)?$', opt_basename):
|
2019-01-03 22:11:33 +08:00
|
|
|
print('ERROR: Unexpected opt name: ' + opt_basename, file=sys.stderr)
|
2016-03-25 07:19:26 +08:00
|
|
|
sys.exit(1)
|
2019-05-12 12:55:09 +08:00
|
|
|
opt_basename = 'opt'
|
2016-03-25 07:19:26 +08:00
|
|
|
|
2019-07-12 04:14:22 +08:00
|
|
|
test_paths = []
|
|
|
|
for test in args.tests:
|
|
|
|
if not glob.glob(test):
|
|
|
|
print('WARNING: Test file \'%s\' was not found. Ignoring it.' % (test,), file=sys.stderr)
|
|
|
|
continue
|
|
|
|
test_paths.append(test)
|
|
|
|
|
2019-03-05 18:44:37 +08:00
|
|
|
for test in test_paths:
|
2016-03-25 07:19:26 +08:00
|
|
|
if args.verbose:
|
2019-01-03 22:11:33 +08:00
|
|
|
print('Scanning for RUN lines in test file: %s' % (test,), file=sys.stderr)
|
2016-03-25 07:19:26 +08:00
|
|
|
with open(test) as f:
|
|
|
|
input_lines = [l.rstrip() for l in f]
|
|
|
|
|
2016-12-30 03:32:34 +08:00
|
|
|
raw_lines = [m.group(1)
|
2018-01-30 08:40:05 +08:00
|
|
|
for m in [common.RUN_LINE_RE.match(l) for l in input_lines] if m]
|
2016-12-30 03:32:34 +08:00
|
|
|
run_lines = [raw_lines[0]] if len(raw_lines) > 0 else []
|
|
|
|
for l in raw_lines[1:]:
|
2016-12-30 04:05:51 +08:00
|
|
|
if run_lines[-1].endswith("\\"):
|
|
|
|
run_lines[-1] = run_lines[-1].rstrip("\\") + " " + l
|
|
|
|
else:
|
|
|
|
run_lines.append(l)
|
2016-12-30 03:32:34 +08:00
|
|
|
|
2016-03-25 07:19:26 +08:00
|
|
|
if args.verbose:
|
2019-01-03 22:11:33 +08:00
|
|
|
print('Found %d RUN lines:' % (len(run_lines),), file=sys.stderr)
|
2016-03-25 07:19:26 +08:00
|
|
|
for l in run_lines:
|
2019-01-03 22:11:33 +08:00
|
|
|
print(' RUN: ' + l, file=sys.stderr)
|
2016-03-25 07:19:26 +08:00
|
|
|
|
|
|
|
prefix_list = []
|
|
|
|
for l in run_lines:
|
|
|
|
(tool_cmd, filecheck_cmd) = tuple([cmd.strip() for cmd in l.split('|', 1)])
|
[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
|
|
|
common.verify_filecheck_prefixes(filecheck_cmd)
|
2017-06-13 01:44:30 +08:00
|
|
|
if not tool_cmd.startswith(opt_basename + ' '):
|
2019-01-03 22:11:33 +08:00
|
|
|
print('WARNING: Skipping non-%s RUN line: %s' % (opt_basename, l), file=sys.stderr)
|
2016-03-25 07:19:26 +08:00
|
|
|
continue
|
|
|
|
|
|
|
|
if not filecheck_cmd.startswith('FileCheck '):
|
2019-01-03 22:11:33 +08:00
|
|
|
print('WARNING: Skipping non-FileChecked RUN line: ' + l, file=sys.stderr)
|
2016-03-25 07:19:26 +08:00
|
|
|
continue
|
|
|
|
|
2017-06-13 01:44:30 +08:00
|
|
|
tool_cmd_args = tool_cmd[len(opt_basename):].strip()
|
2016-03-25 07:19:26 +08:00
|
|
|
tool_cmd_args = tool_cmd_args.replace('< %s', '').replace('%s', '').strip()
|
|
|
|
|
2018-01-30 08:40:05 +08:00
|
|
|
check_prefixes = [item for m in common.CHECK_PREFIX_RE.finditer(filecheck_cmd)
|
2017-01-14 17:39:35 +08:00
|
|
|
for item in m.group(1).split(',')]
|
2016-03-25 07:19:26 +08:00
|
|
|
if not check_prefixes:
|
|
|
|
check_prefixes = ['CHECK']
|
|
|
|
|
|
|
|
# FIXME: We should use multiple check prefixes to common check lines. For
|
|
|
|
# now, we just ignore all but the last.
|
|
|
|
prefix_list.append((check_prefixes, tool_cmd_args))
|
|
|
|
|
|
|
|
func_dict = {}
|
|
|
|
for prefixes, _ in prefix_list:
|
|
|
|
for prefix in prefixes:
|
|
|
|
func_dict.update({prefix: dict()})
|
2017-06-13 01:44:30 +08:00
|
|
|
for prefixes, opt_args in prefix_list:
|
2016-03-25 07:19:26 +08:00
|
|
|
if args.verbose:
|
2019-01-03 22:11:33 +08:00
|
|
|
print('Extracted opt cmd: ' + opt_basename + ' ' + opt_args, file=sys.stderr)
|
|
|
|
print('Extracted FileCheck prefixes: ' + str(prefixes), file=sys.stderr)
|
2016-03-25 07:19:26 +08:00
|
|
|
|
2018-01-30 08:40:05 +08:00
|
|
|
raw_tool_output = common.invoke_tool(args.opt_binary, opt_args, test)
|
|
|
|
common.build_function_body_dictionary(
|
2018-02-10 13:01:33 +08:00
|
|
|
common.OPT_FUNCTION_RE, common.scrub_body, [],
|
|
|
|
raw_tool_output, prefixes, func_dict, args.verbose)
|
2016-03-25 07:19:26 +08:00
|
|
|
|
|
|
|
is_in_function = False
|
|
|
|
is_in_function_start = False
|
|
|
|
prefix_set = set([prefix for prefixes, _ in prefix_list for prefix in prefixes])
|
|
|
|
if args.verbose:
|
2019-01-03 22:11:33 +08:00
|
|
|
print('Rewriting FileCheck prefixes: %s' % (prefix_set,), file=sys.stderr)
|
2016-03-25 07:19:26 +08:00
|
|
|
output_lines = []
|
|
|
|
output_lines.append(autogenerated_note)
|
|
|
|
|
|
|
|
for input_line in input_lines:
|
|
|
|
if is_in_function_start:
|
|
|
|
if input_line == '':
|
|
|
|
continue
|
|
|
|
if input_line.lstrip().startswith(';'):
|
2018-01-30 08:40:05 +08:00
|
|
|
m = common.CHECK_RE.match(input_line)
|
2016-03-25 07:19:26 +08:00
|
|
|
if not m or m.group(1) not in prefix_set:
|
|
|
|
output_lines.append(input_line)
|
|
|
|
continue
|
|
|
|
|
|
|
|
# Print out the various check lines here.
|
2018-04-05 17:30:42 +08:00
|
|
|
common.add_ir_checks(output_lines, ';', prefix_list, func_dict, func_name)
|
2016-03-25 07:19:26 +08:00
|
|
|
is_in_function_start = False
|
|
|
|
|
|
|
|
if is_in_function:
|
2018-01-30 08:40:05 +08:00
|
|
|
if common.should_add_line_to_output(input_line, prefix_set):
|
2016-03-25 07:19:26 +08:00
|
|
|
# This input line of the function body will go as-is into the output.
|
2016-03-28 04:43:02 +08:00
|
|
|
# Except make leading whitespace uniform: 2 spaces.
|
2018-01-30 08:40:05 +08:00
|
|
|
input_line = common.SCRUB_LEADING_WHITESPACE_RE.sub(r' ', input_line)
|
2016-03-25 07:19:26 +08:00
|
|
|
output_lines.append(input_line)
|
|
|
|
else:
|
|
|
|
continue
|
|
|
|
if input_line.strip() == '}':
|
|
|
|
is_in_function = False
|
|
|
|
continue
|
|
|
|
|
2016-04-06 03:50:21 +08:00
|
|
|
# Discard any previous script advertising.
|
|
|
|
if input_line.startswith(ADVERT):
|
2016-03-25 07:19:26 +08:00
|
|
|
continue
|
|
|
|
|
|
|
|
# If it's outside a function, it just gets copied to the output.
|
|
|
|
output_lines.append(input_line)
|
|
|
|
|
|
|
|
m = IR_FUNCTION_RE.match(input_line)
|
|
|
|
if not m:
|
|
|
|
continue
|
2018-01-30 08:40:05 +08:00
|
|
|
func_name = m.group(1)
|
|
|
|
if args.function is not None and func_name != args.function:
|
2016-03-25 07:19:26 +08:00
|
|
|
# When filtering on a specific function, skip all others.
|
|
|
|
continue
|
|
|
|
is_in_function = is_in_function_start = True
|
|
|
|
|
|
|
|
if args.verbose:
|
2019-01-03 22:11:33 +08:00
|
|
|
print('Writing %d lines to %s...' % (len(output_lines), test), file=sys.stderr)
|
2016-03-25 07:19:26 +08:00
|
|
|
|
|
|
|
with open(test, 'wb') as f:
|
2019-01-31 00:15:59 +08:00
|
|
|
f.writelines(['{}\n'.format(l).encode('utf-8') for l in output_lines])
|
2016-03-25 07:19:26 +08:00
|
|
|
|
|
|
|
|
|
|
|
if __name__ == '__main__':
|
|
|
|
main()
|