[PATCH] D117003: [SchedModels][CortexA55] Add ASIMD integer instructioins

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 28 03:05:02 PST 2022


dmgreen added a comment.

I have partially reverted this in 61b616755aced8ed7afc48ffd152f02194b9d201 <https://reviews.llvm.org/rG61b616755aced8ed7afc48ffd152f02194b9d201>. I was trying not to undo the whole thing, but just removed the forwarding paths and some other parts that were making performance worse around the "L" instructions. The rest was honestly making some performance worse too, but some stuff was better and the parts removed seemed to be causing much of the change. We probably need to be more careful going forward that we benchmark on more cpu's, not just the Cortex-A55. The schedule is used by any -mcpu=generic compile, so even if it's a less accurate model of the A55, we may need to strike more of a balance between different cpus until we have a better option.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117003



More information about the llvm-commits mailing list