[PATCH] D128631: [AArch64] Initial sched model for Neoverse N2

[dewen via Phabricator via llvm-commits]

dewen added inline comments.


================
Comment at: llvm/test/tools/llvm-mca/AArch64/Neoverse/N2-sve-instructions.s:7173
+# CHECK-NEXT:  27     11    4.50           *      U     st4b	{ z5.b, z6.b, z7.b, z8.b }, p3, [x17, x16]
+# CHECK-NEXT:  27     11    4.50           *      U     st4d	{ z0.d, z1.d, z2.d, z3.d }, p0, [x0, x0, lsl #3]
+# CHECK-NEXT:  18     11    4.50           *      U     st4d	{ z0.d, z1.d, z2.d, z3.d }, p0, [x0]
----------------
c-rhodes wrote:
> dewen wrote:
> > dewen wrote:
> > > c-rhodes wrote:
> > > > dewen wrote:
> > > > > c-rhodes wrote:
> > > > > > dewen wrote:
> > > > > > > dewen wrote:
> > > > > > > > I found that the throughput information for this instruction (`st4d`) does not match the description in the documentation. Is there a problem with the document? 
> > > > > > > > {F25275327}
> > > > > > > > 
> > > > > > > > I found that the throughput information for this instruction (`st4d`) does not match the description in the documentation. Is there a problem with the document? 
> > > > > > > > {F25275327}
> > > > > > > > 
> > > > > > > 
> > > > > > > @Cullen Rhodes
> > > > > > > > I found that the throughput information for this instruction (`st4d`) does not match the description in the documentation. Is there a problem with the document? 
> > > > > > > > {F25275327}
> > > > > > > > 
> > > > > > > 
> > > > > > > @Cullen Rhodes
> > > > > > 
> > > > > > I've seen your comment, I don't have an immediate answer but will check.
> > > > > Hey, has this comment come to a conclusion? @c-rhodes 
> > > > > Hey, has this comment come to a conclusion? @c-rhodes 
> > > > 
> > > > The throughput for the ST4 instructions here is incorrect as you pointed out, it should be 9. I noticed ST2 doesn't look correct either. Thanks for reporting, we'll get it fixed.
> > > Okay, look forward to your fix.
> > Hi, when are you planning to fix this? I'm looking forward to it. @c-rhodes
> > Hi, when are you planning to fix this? I'm looking forward to it. @c-rhodes
> 
> I'm no longer working in this area, someone from my team will fix it but it's not urgent enough to drop everything they're currently doing unless there's something I'm not aware of. Is this blocking you somehow?
Haha, I've had a similar problem recently, but I don't know how to solve it, so it's urgent.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128631