[PATCH] D78105: [CSInfo][ISEL] Call site info generation support for Mips
David Stenberg via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed May 13 03:43:52 PDT 2020
dstenb added a comment.
I have a comment about the CHECK/PARSER test cases. Otherwise this looks good to me.
================
Comment at: llvm/test/CodeGen/Mips/call-site-info-output.ll:34
+; CHECK-NEXT: arg: 2, reg: '$a2'
+;; Verify that we are able to parse output mir and that we are getting the same result.
+; PARSER: name: fn2
----------------
This comment says that we parse output MIR, but the PARSER commands take IR as input?
```
RUN: llc -mtriple=mips-linux-gnu -emit-call-site-info %s -stop-after=finalize-isel -o -| FileCheck %s --check-prefix=PARSER
```
To test MIR parsing, perhaps we can just combine the two RUN lines into something like this?
```
llc %s -stop-after=finalize-isel -o - | llc -run-pass=finalize-isel -o - | Filecheck %s
```
And if we keep two separate RUN invocations, the `CHECK` and `PARSER` are identical as far as I can tell, so can we combine them to one group then?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78105/new/
https://reviews.llvm.org/D78105
More information about the llvm-commits
mailing list