[PATCH] D71266: [ARM] Return a number of micro-ops for vlldm/vlstm

[Dave Green via Phabricator via llvm-commits]

dmgreen added a comment.

I think that this function is only considering Itineraries? When a MISchedule isn't present? They are a bit Legacy now, and I wouldn't expect any cpu with these instructions to be using them.

It looks like in the instruction definitions, the VLSTM and VLLDM have IIC_fpLoad_m and IIC_fpStore_m. That would appear to be where the "multi-uops" is coming from. Can we just remove that? Use no itineraries instead? At the very least, like you said, the VLLDM and VLSTM are lazy so will not often not infer a cost at the point of them being ran. Maybe a NOP, if there is one? But I think None would be fine too.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71266/new/

https://reviews.llvm.org/D71266