[PATCH] D152333: update_mir_test_checks.py doesn't separate different prefix checks #63112

Eddie Phillips via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 7 11:06:00 PDT 2023


eddiep24 added a comment.

The lines at the top were being made because the should_add_line_to_output(input_line, prefix_set) function was skipping check lines but not the comments.  Thus, the comments were getting pushed to the top.  My solution was to ignore comments in the same fashion and use the previous line in the output_lines array to identify the correct indentation.  Thanks for the feedback on the previous diff :).  Attached are the diffs for the tests you recommended. They worked out nicely.
 F27857006: legalize-add-v256.diff <https://reviews.llvm.org/F27857006>

F27857005: legalize-add-v512.diff <https://reviews.llvm.org/F27857005>

F27857004: legalize-add.diff <https://reviews.llvm.org/F27857004>


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

https://reviews.llvm.org/D152333



More information about the llvm-commits mailing list