[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