[PATCH] D69998: [MacroFusion] Create the missing artificial edges if there are more than 2 SU fused.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 6 02:38:54 PST 2019


fhahn added a comment.

In D69998#1772106 <https://reviews.llvm.org/D69998#1772106>, @steven.zhang wrote:

> Hmm, we are putting a bomb here if someone want to get extends :P But I agree with you that as we cannot test this patch now, it is NOT the best time to fix it.


One possible way to deal with that would be to add an assertion that we chain at most 2 instructions together here, with a comment what the issue will be with chains longer than 2 instructions.


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

https://reviews.llvm.org/D69998





More information about the llvm-commits mailing list