[PATCH] D71266: [ARM] Return a number of micro-ops for vlldm/vlstm
Dave Green via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Dec 11 01:17:01 PST 2019
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
More information about the llvm-commits
mailing list