[PATCH] D32037: Change the testcase tail-merge-after-mbp.ll to tail-merge-after-mbp.mir

Geoff Berry via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 13 11:41:32 PDT 2017


gberry added a reviewer: MatzeB.
gberry added a comment.

I believe a lot of the IR in this test case is superfluous (as well as some of the MIR property flags like hasStackMap, etc.).  Could you try cutting it down to only the necessary IR and MIR to make it more robust against future changes?
I've added Matthias as a reviewer since he is more familiar with the current state of the MIR parser.


Repository:
  rL LLVM

https://reviews.llvm.org/D32037





More information about the llvm-commits mailing list