[PATCH] D71217: Fix incorrect logic in maintaining the side-effect of compiler generated outliner functions

Jin Lin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 4 13:30:20 PST 2020


jinlin added a comment.

In D71217#1906007 <https://reviews.llvm.org/D71217#1906007>, @aemerson wrote:

> Hi, I just tried out this patch locally and I'm seeing failures running the tests:
>
>   Failing Tests (3):
>        LLVM :: CodeGen/AArch64/machine-outliner-cfi.mir
>        LLVM :: CodeGen/AArch64/machine-outliner-noreturn-save-lr.mir
>        LLVM :: CodeGen/AArch64/machine-outliner-side-effect.mir


I am sorry to post the the incorrect version of the file MachineOutliner in the last diff.

When I compared the last diff (10)  with diff 9, I found that the file MachineOutliner is in the old version. This morning I tried to use git rebase master in my local branch, somehow this file was changed when the conflict happened.

I am working on the revert now.


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

https://reviews.llvm.org/D71217





More information about the llvm-commits mailing list