[PATCH] D79747: Implementation of Simple Outliner Module Pass

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 3 12:38:31 PDT 2020


fhahn added a comment.

I think it would be great to add more details about the benefit of the outlining to the description. For the example in the test case, outlining seems quite heavy handed. Wouldn't it be simpler and faster in those cases to just duplicate the return block?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79747





More information about the llvm-commits mailing list