Skip to content

Commit

Permalink
update_mir_test_checks.py - separate different prefix checks
Browse files Browse the repository at this point in the history
Matches behaviour in update_llc_test_checks.py etc.

Fixes #63112

Differential Revision: https://reviews.llvm.org/D152333
  • Loading branch information
eddiep24 authored and RKSimon committed Jul 6, 2023
1 parent b4049b4 commit 948375f
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@ define void @zero128() nounwind ssp {
; NOAVX-NEXT: [[MOV64rm:%[0-9]+]]:gr64 = MOV64rm $rip, 1, $noreg, target-flags(x86-gotpcrel) @z, $noreg :: (load (s64) from got, align 16)
; NOAVX-NEXT: MOVAPSmr killed [[MOV64rm]], 1, $noreg, 0, $noreg, killed [[V_SET0_]] :: (store (s128) into @z)
; NOAVX-NEXT: RET 0
;
; AVX-LABEL: name: zero128
; AVX: bb.0 (%ir-block.0):
; AVX-NEXT: [[V_SET0_:%[0-9]+]]:vr128 = V_SET0
Expand Down
11 changes: 6 additions & 5 deletions llvm/utils/update_mir_test_checks.py
Original file line number Diff line number Diff line change
Expand Up @@ -235,9 +235,10 @@ def add_checks_for_function(
break
if not func_dict[prefix][func_name]:
continue
# if printed_prefixes:
# # Add some space between different check prefixes.
# output_lines.append('')
if printed_prefixes:
# Add some space between different check prefixes.
indent = len(output_lines[-1]) - len(output_lines[-1].lstrip(" "))
output_lines.append(" "*indent + ";")
printed_prefixes.add(prefix)
log("Adding {} lines for {}".format(prefix, func_name), args.verbose)
add_check_lines(
Expand Down Expand Up @@ -333,9 +334,9 @@ def mangle_vreg(opcode, current_names):


def should_add_line_to_output(input_line, prefix_set):
# Skip any check lines that we're handling.
# Skip any check lines that we're handling as well as comments
m = common.CHECK_RE.match(input_line)
if m and m.group(1) in prefix_set:
if (m and m.group(1) in prefix_set) or re.search("^[ \t]*;", input_line):
return False
return True

Expand Down

0 comments on commit 948375f

Please sign in to comment.