[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