[PATCH] D71180: [NFC][MacroFusion] Adding the assertion if someone want to fuse more than 2 instructions.
qshanz via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 9 18:13:19 PST 2019
steven.zhang marked an inline comment as done.
steven.zhang added inline comments.
================
Comment at: llvm/lib/CodeGen/MacroFusion.cpp:80
+ assert(hasLessThanNumFused(FirstSU, 2) &&
+ "Chain two instructions together at most");
+
----------------
fhahn wrote:
> nit: maybe it would be slightly better to put a whole sentence here, e.g. something like 'Currently we only support chaining together 2 instructions.'
ok, I will update it when commit it.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71180/new/
https://reviews.llvm.org/D71180
More information about the llvm-commits
mailing list