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

Mark Lacey via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 16 08:22:17 PST 2022


rudkx marked an inline comment as done.
rudkx added inline comments.


================
Comment at: llvm/test/Transforms/PartialInlining/recursive_partial_inlining.ll:1
+; NOTE: Assertions have been autogenerated by utils/update_test_checks.py
+; RUN: opt -partial-inliner -skip-partial-inlining-cost-analysis -S < %s | FileCheck %s
----------------
efriedma wrote:
> update_test_checks.py has a flag --include-generated-funcs you might want to use here.
I tried using that flag, and unfortunately it seems to be generating check lines that FileCheck fails to match. The check lines look fine to me, so it may be a FileCheck bug. I've had similar issues in the past with hand-written lines where FileCheck seems to either get hung-up on trailing text on a line, or is having a problem with variable captures that look identical but somehow it doesn't think match.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136383



More information about the llvm-commits mailing list