[PATCH] D77748: [CodeGen] Allow partial tail duplication in Machine Block Placement.

Hongtao Yu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 8 11:57:26 PDT 2020


hoyFB created this revision.
Herald added subscribers: llvm-commits, hiraditya.
Herald added a project: LLVM.
hoyFB edited the summary of this revision.
hoyFB added a reviewer: wenlei.

A count profile may affect tail duplication's heuristic causing a block to be duplicated in only a part of its predecessors. This is not allowed in the Machine Block Placement pass where an assert will go off. I'm removing the assert and making the optimization bail out when such case happens.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D77748

Files:
  llvm/lib/CodeGen/MachineBlockPlacement.cpp
  llvm/test/CodeGen/X86/tail-dup-partial.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D77748.256083.patch
Type: text/x-patch
Size: 4614 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200408/01008220/attachment.bin>


More information about the llvm-commits mailing list