[PATCH] D103289: A post-processing for BFI inference

[David Li via Phabricator via llvm-commits]

davidxl added a comment.

thanks for working on this issue. A high level question -- is it possible to do the fix up on a per (irreducible) loop basis?



================
Comment at: llvm/test/Transforms/SampleProfile/profile-correlation-irreducible-loops.ll:1
+; RUN: opt < %s -passes=pseudo-probe,sample-profile -sample-profile-file=%S/Inputs/profile-correlation-irreducible-loops.prof | opt -analyze -block-freq -enable-new-pm=0  -use-iterative-bfi-inference | FileCheck %s
+; RUN: opt < %s -passes=pseudo-probe,sample-profile -sample-profile-file=%S/Inputs/profile-correlation-irreducible-loops.prof -S | FileCheck %s --check-prefix=CHECK2
----------------
why -enable-new-pm = 0?


================
Comment at: llvm/test/Transforms/SampleProfile/profile-correlation-irreducible-loops.ll:10
+
+; Function Attrs: noinline nounwind uwtable
+define dso_local i32 @yyparse_1() #0 {
----------------
It will be helpful to draw a simple text art CFG to demonstrate the expected bb counts.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103289