[PATCH] D71180: [NFC][MacroFusion] Adding the assertion if someone want to fuse more than 2 instructions.
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 9 11:11:43 PST 2019
fhahn accepted this revision.
fhahn added a comment.
This revision is now accepted and ready to land.
LGTM, thanks!
================
Comment at: llvm/lib/CodeGen/MacroFusion.cpp:80
+ assert(hasLessThanNumFused(FirstSU, 2) &&
+ "Chain two instructions together at most");
+
----------------
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.'
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71180/new/
https://reviews.llvm.org/D71180
More information about the llvm-commits
mailing list