[PATCH] D136383: [PartialInlining] Enable recursive partial inlining.

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks added a comment.

are you see that this is beneficial in practice?



================
Comment at: llvm/test/Transforms/PartialInlining/recursive_partial_inlining.ll:2
+; NOTE: Assertions have been autogenerated by utils/update_test_checks.py
+; RUN: opt -partial-inliner -skip-partial-inlining-cost-analysis --simplifycfg -S < %s | FileCheck %s
+define void @_Z26recursive_partial_inliningi(i32 noundef %i) local_unnamed_addr {
----------------
can you remove the simplifycfg run so we can more accurately see what the pass is doing?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136383