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

Wei Mi via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 13 13:56:28 PDT 2017


On Thu, Apr 13, 2017 at 1:34 PM, Geoff Berry via Phabricator
<reviews at reviews.llvm.org> wrote:
> gberry added a comment.
>
> Sorry if I wasn't clear.  I'm not suggesting you change what is being tested, just that you remove code from the test that doesn't impact it.  For example, I believe you can get rid of all of the IR code if you also remove the MemoryOperands info from the MIR (the :: (load 8 from xxx) stuff).

Thanks. I removed some code per your suggestion, but I am not familiar
with MIR enough to know whether I have done it throughly.

>
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D32037
>
>
>


More information about the llvm-commits mailing list