[llvm] [TailDuplicator] Add a limit on the size of predecessors (PR #78582)

Quentin Dian via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 22 20:02:02 PST 2024


DianQK wrote:

> > Hmm, CodeGen is not an area I know much about. Could you explain that? What does profile information refer to, the information provided by the PGO?
> 
> Right. Disable tail duplication for blocks with 8 predecessors may hurt performance of some applications. Duplicate blocks into hot predecessors only can still give you the benefit of tail duplication, and at the same time limit the number of duplication. MachineBlockPlacement does the same thing in the late tail duplication (embedded in the MBP).

But the currently given example does not contain profile information. I don't think this is a solution to the current problem to be considered.

https://github.com/llvm/llvm-project/pull/78582


More information about the llvm-commits mailing list