[PATCH] D40030: [AArch64][TableGen] Skip tied result operands for InstAlias

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer accepted this revision.
SjoerdMeijer added a comment.
This revision is now accepted and ready to land.

Looks like a straightforward fix of https://reviews.llvm.org/D29219 to me. But perhaps wait a little bit with committing, just in case @rengolin who did the original review has some more comments.



================
Comment at: utils/TableGen/AsmWriterEmitter.cpp:823
       unsigned NumMIOps = 0;
-      for (auto &Operand : CGA.ResultOperands)
-        NumMIOps += Operand.getMINumOperands();
+      for (unsigned i = 0, e = CGA.ResultInst->Operands.size(); i != e; ++i)
+        NumMIOps += CGA.ResultInst->Operands[i].MINumOperands;
----------------
Nit: do we need to set/use 'e' here?


https://reviews.llvm.org/D40030