Skip to content

Commit

Permalink
llvm-reduce: Speculative fix for windows build bot
Browse files Browse the repository at this point in the history
I'm guessing grep wasn't matching the backtick in the message for some
reason.
  • Loading branch information
arsenm committed Jan 3, 2023
1 parent 0008afc commit 84e888e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion llvm/test/tools/llvm-reduce/command-line-behavior.test
Expand Up @@ -3,7 +3,7 @@
# RUN: llvm-reduce | grep "LLVM automatic testcase reducer"

# Don't require any other arguments for --print-delta-passes
# RUN: llvm-reduce --print-delta-passes | grep "Delta passes (pass to \`--delta-passes=\` as a comma separated list)"
# RUN: llvm-reduce --print-delta-passes | grep "Delta passes"

# Missing test input
# RUN: not llvm-reduce --test FileCheck 2>&1 | grep "error: reduction testcase positional argument must be specified"
Expand Down

1 comment on commit 84e888e

@alvinhochun
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps you should consider using FileCheck instead. https://llvm.org/docs/TestingGuide.html#writing-new-regression-tests states: "The usage of grep in RUN lines is deprecated".

Please sign in to comment.