[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