[PATCH] D135579: utils/update_mir_test_checks.py: allow checking fixedStack in .mir files

[Nicolai Hähnle via Phabricator via llvm-commits]

nhaehnle added a comment.

The overall approach seems good to me.



================
Comment at: llvm/test/tools/UpdateTestChecks/update_mir_test_checks/Inputs/print-stack-first.mir:3-5
+# Note that this file isn't a test in itself (Inputs/ is excluded from lit's
+# test discovery). Instead, it is an input to the print-stack.test test that
+# verifies the --print-fixed-stack option of update_mir_test_checks.py.
----------------
This comment isn't really necessary.


================
Comment at: llvm/utils/update_mir_test_checks.py:247
 def add_check_lines(test, output_lines, prefix, func_name, single_bb,
-                    func_body):
+                    func_info :FunctionInfo, args):
+    func_body = func_info.body.splitlines()
----------------
I don't think we really have a Python coding style guide, but this should probably be `func_info: FunctionInfo`


================
Comment at: llvm/utils/update_mir_test_checks.py:269
+            filecheck_directive = check + '-NEXT'
+            first_check = False
+            output_lines.append('{}: {}'.format(filecheck_directive, stack_line))
----------------
This assignment seems pointless?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D135579/new/

https://reviews.llvm.org/D135579