2019-11-26 19:40:52 +01:00
|
|
|
#!/usr/bin/env python3
|
2018-04-06 14:36:27 +02:00
|
|
|
|
|
|
|
"""A script to generate FileCheck statements for 'opt' analysis tests.
|
|
|
|
|
|
|
|
This script is a utility to update LLVM opt analysis test cases with new
|
|
|
|
FileCheck patterns. It can either update all of the tests in the file or
|
|
|
|
a single test function.
|
|
|
|
|
|
|
|
Example usage:
|
|
|
|
$ update_analyze_test_checks.py --opt=../bin/opt test/foo.ll
|
|
|
|
|
|
|
|
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 -analyze -cost-model -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!
|
|
|
|
"""
|
|
|
|
|
2019-01-03 15:11:33 +01:00
|
|
|
from __future__ import print_function
|
|
|
|
|
2018-04-06 14:36:27 +02:00
|
|
|
import argparse
|
2019-03-05 11:44:37 +01:00
|
|
|
import glob
|
2018-04-06 14:36:27 +02:00
|
|
|
import itertools
|
|
|
|
import os # Used to advertise this file's name ("autogenerated_note").
|
|
|
|
import string
|
|
|
|
import subprocess
|
|
|
|
import sys
|
|
|
|
import tempfile
|
|
|
|
import re
|
|
|
|
|
|
|
|
from UpdateTestChecks import common
|
|
|
|
|
|
|
|
ADVERT = '; NOTE: Assertions have been autogenerated by '
|
|
|
|
|
|
|
|
# RegEx: this is where the magic happens.
|
|
|
|
|
|
|
|
IR_FUNCTION_RE = re.compile('^\s*define\s+(?:internal\s+)?[^@]*@([\w-]+)\s*\(')
|
|
|
|
|
|
|
|
def main():
|
|
|
|
from argparse import RawTextHelpFormatter
|
|
|
|
parser = argparse.ArgumentParser(description=__doc__, formatter_class=RawTextHelpFormatter)
|
|
|
|
parser.add_argument('--opt-binary', default='opt',
|
|
|
|
help='The opt binary used to generate the test case')
|
|
|
|
parser.add_argument(
|
|
|
|
'--function', help='The function in the test file to update')
|
|
|
|
parser.add_argument('tests', nargs='+')
|
2019-11-20 14:19:48 +01:00
|
|
|
args = common.parse_commandline_args(parser)
|
2018-04-06 14:36:27 +02:00
|
|
|
|
2019-08-07 16:44:50 +02:00
|
|
|
script_name = os.path.basename(__file__)
|
|
|
|
autogenerated_note = (ADVERT + 'utils/' + script_name)
|
2018-04-06 14:36:27 +02:00
|
|
|
|
|
|
|
opt_basename = os.path.basename(args.opt_binary)
|
|
|
|
if (opt_basename != "opt"):
|
2019-08-07 16:44:50 +02:00
|
|
|
common.error('Unexpected opt name: ' + opt_basename)
|
2018-04-06 14:36:27 +02:00
|
|
|
sys.exit(1)
|
|
|
|
|
2019-03-05 11:44:37 +01:00
|
|
|
test_paths = [test for pattern in args.tests for test in glob.glob(pattern)]
|
|
|
|
for test in test_paths:
|
2018-04-06 14:36:27 +02:00
|
|
|
with open(test) as f:
|
|
|
|
input_lines = [l.rstrip() for l in f]
|
|
|
|
|
2019-08-07 16:44:50 +02:00
|
|
|
first_line = input_lines[0] if input_lines else ""
|
|
|
|
if 'autogenerated' in first_line and script_name not in first_line:
|
|
|
|
common.warn("Skipping test which wasn't autogenerated by " + script_name + ": " + test)
|
|
|
|
continue
|
|
|
|
|
|
|
|
if args.update_only:
|
|
|
|
if not first_line or 'autogenerated' not in first_line:
|
|
|
|
common.warn("Skipping test which isn't autogenerated: " + test)
|
|
|
|
continue
|
|
|
|
|
2019-12-02 11:50:23 +01:00
|
|
|
run_lines = common.find_run_lines(test, input_lines)
|
2018-04-06 14:36:27 +02:00
|
|
|
prefix_list = []
|
|
|
|
for l in run_lines:
|
2019-08-07 16:44:50 +02:00
|
|
|
if '|' not in l:
|
|
|
|
common.warn('Skipping unparseable RUN line: ' + l)
|
|
|
|
continue
|
|
|
|
|
2018-04-06 14:36:27 +02:00
|
|
|
(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-29 19:41:00 +02:00
|
|
|
common.verify_filecheck_prefixes(filecheck_cmd)
|
2018-04-06 14:36:27 +02:00
|
|
|
|
|
|
|
if not tool_cmd.startswith(opt_basename + ' '):
|
2019-08-07 16:44:50 +02:00
|
|
|
common.warn('WSkipping non-%s RUN line: %s' % (opt_basename, l))
|
2018-04-06 14:36:27 +02:00
|
|
|
continue
|
|
|
|
|
|
|
|
if not filecheck_cmd.startswith('FileCheck '):
|
2019-08-07 16:44:50 +02:00
|
|
|
common.warn('Skipping non-FileChecked RUN line: ' + l)
|
2018-04-06 14:36:27 +02:00
|
|
|
continue
|
|
|
|
|
|
|
|
tool_cmd_args = tool_cmd[len(opt_basename):].strip()
|
|
|
|
tool_cmd_args = tool_cmd_args.replace('< %s', '').replace('%s', '').strip()
|
|
|
|
|
|
|
|
check_prefixes = [item for m in common.CHECK_PREFIX_RE.finditer(filecheck_cmd)
|
|
|
|
for item in m.group(1).split(',')]
|
|
|
|
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()})
|
|
|
|
for prefixes, opt_args in prefix_list:
|
2019-12-02 11:50:23 +01:00
|
|
|
common.debug('Extracted opt cmd:', opt_basename, opt_args, file=sys.stderr)
|
|
|
|
common.debug('Extracted FileCheck prefixes:', str(prefixes), file=sys.stderr)
|
2018-04-06 14:36:27 +02:00
|
|
|
|
|
|
|
raw_tool_outputs = common.invoke_tool(args.opt_binary, opt_args, test)
|
|
|
|
|
|
|
|
# Split analysis outputs by "Printing analysis " declarations.
|
|
|
|
for raw_tool_output in re.split(r'Printing analysis ', raw_tool_outputs):
|
|
|
|
common.build_function_body_dictionary(
|
|
|
|
common.ANALYZE_FUNCTION_RE, common.scrub_body, [],
|
[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-10 19:08:21 +02:00
|
|
|
raw_tool_output, prefixes, func_dict, args.verbose, False)
|
2018-04-06 14:36:27 +02:00
|
|
|
|
|
|
|
is_in_function = False
|
|
|
|
is_in_function_start = False
|
|
|
|
prefix_set = set([prefix for prefixes, _ in prefix_list for prefix in prefixes])
|
2019-12-02 11:50:23 +01:00
|
|
|
common.debug('Rewriting FileCheck prefixes:', str(prefix_set), file=sys.stderr)
|
2018-04-06 14:36:27 +02: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(';'):
|
|
|
|
m = common.CHECK_RE.match(input_line)
|
|
|
|
if not m or m.group(1) not in prefix_set:
|
|
|
|
output_lines.append(input_line)
|
|
|
|
continue
|
|
|
|
|
|
|
|
# Print out the various check lines here.
|
|
|
|
common.add_analyze_checks(output_lines, ';', prefix_list, func_dict, func_name)
|
|
|
|
is_in_function_start = False
|
|
|
|
|
|
|
|
if is_in_function:
|
|
|
|
if common.should_add_line_to_output(input_line, prefix_set):
|
|
|
|
# This input line of the function body will go as-is into the output.
|
|
|
|
# Except make leading whitespace uniform: 2 spaces.
|
|
|
|
input_line = common.SCRUB_LEADING_WHITESPACE_RE.sub(r' ', input_line)
|
|
|
|
output_lines.append(input_line)
|
|
|
|
else:
|
|
|
|
continue
|
|
|
|
if input_line.strip() == '}':
|
|
|
|
is_in_function = False
|
|
|
|
continue
|
|
|
|
|
|
|
|
# Discard any previous script advertising.
|
|
|
|
if input_line.startswith(ADVERT):
|
|
|
|
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
|
|
|
|
func_name = m.group(1)
|
|
|
|
if args.function is not None and func_name != args.function:
|
|
|
|
# When filtering on a specific function, skip all others.
|
|
|
|
continue
|
|
|
|
is_in_function = is_in_function_start = True
|
|
|
|
|
2019-12-02 11:50:23 +01:00
|
|
|
common.debug('Writing %d lines to %s...' % (len(output_lines), test))
|
2018-04-06 14:36:27 +02:00
|
|
|
|
|
|
|
with open(test, 'wb') as f:
|
2019-01-30 17:15:59 +01:00
|
|
|
f.writelines(['{}\n'.format(l).encode('utf-8') for l in output_lines])
|
2018-04-06 14:36:27 +02:00
|
|
|
|
|
|
|
|
|
|
|
if __name__ == '__main__':
|
|
|
|
main()
|