[PATCH] D71027: Support repeated machine outlining

Mon Dec 9 09:37:55 PST 2019

jinlin added a comment.

In D71027#1774949 <https://reviews.llvm.org/D71027#1774949>, @tellenbach wrote:

> Okay, thanks for clarifying.
>
> In D71027#1772142 <https://reviews.llvm.org/D71027#1772142>, @jinlin wrote:
>
> > In D71027 <https://reviews.llvm.org/D71027>, I have also fixed incorrect logic in maintaining the side-effect information for outlined function at line 1295. This bug shows up during repeated machine outliner.
>
>
> Would it be possible to separate both patches? The fix can then be reviewed independently from repeated outlining itself.

Sure. I have updated the patch. Please note that the repeating machine outliner may not work correctly without fixing the logic at line 1265.

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

https://reviews.llvm.org/D71027