[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