[PATCH] D111531: Fix minor deficiency in MachineSink.

[Stephen Tozer via Phabricator via llvm-commits]

StephenTozer added inline comments.


================
Comment at: llvm/test/CodeGen/RISCV/MachineSink-implicit-x0.mir:35
+...
+---
----------------
markus wrote:
> StephenTozer wrote:
> > I've just run the test, and it seems that this last line causes a parse error for llc; otherwise the test seems to work as written.
> Hmm, what exactly are you running then? 
> When I run
> ```
> ./bin/llvm-lit ../llvm/test/CodeGen/RISCV/MachineSink-implicit-x0.mir
> ```
> it passes without issues.
The command:
```
python .\build\Release\bin\llvm-lit.py llvm\test\CodeGen\RISCV\MachineSink-implicit-x0.mir
```
fails with:
```
$ ":" "RUN: at line 1"
$ "d:\upstream-llvm\build\release\bin\llc.exe" "-mtriple=riscv32" "D:\upstream-llvm\llvm\test\CodeGen\RISCV\MachineSink-implicit-x0.mir" "-run-pass=machine-sink" "-o" "-"
# command stderr:
error: YAML:35:1: not a mapping
---
^~~
```

Deleting the dashes causes it to pass. This is with python 3.9.7 and an up-to-date build of LLVM from trunk (82b74363a9).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111531