[PATCH] D62617: [MIR-Canon] Fixing case where MachineFunction is empty.

Puyan Lotfi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 29 12:27:24 PDT 2019


plotfi marked an inline comment as done.
plotfi added inline comments.


================
Comment at: llvm/test/CodeGen/MIR/AArch64/print-parse-overloaded-intrinsics.mir:5-6
 
+# RUN: llc -mtriple aarch64-- -run-pass mir-canonicalizer %s -o %t \
+# RUN:   -verify-machineinstrs
+
----------------
arsenm wrote:
> I don't see how the test is related?
You have a good point. I will put together a separate test. This one just needs to be (empty MIR, but the IR part has a function):

```
--- |
  define i32 @foo() {
    ret i32 0
  }
...
```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62617





More information about the llvm-commits mailing list