[PATCH] D70066: [MacroFusion] Limit the max fused number as 2 to reduce the dependency

[qshanz via Phabricator via llvm-commits]

steven.zhang added a comment.

In D70066#1921272 <https://reviews.llvm.org/D70066#1921272>, @fhahn wrote:

> I realized that I had a small un-submitted comment here.....Please also update the documentation for shouldScheduleAdjacent to clarify that it won't be called if DepMI is already part of a pair.


Err.. But it is a callback...  I didn't find a good place to give this kind of message for shouldScheduleAdjacent. Any suggestions ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70066