[PATCH] D137841: [X86] Reduce unnecessary instregex for AlderlakeP schedule model

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 11 09:08:02 PST 2022


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

Thanks - the instregex -> instrs diffs LGTM

Do you have any suggestions for scheduler classes that we could add/change to help reduce the number of overrides?

After finishing my cleanup of the the conversions I want to look at the old x86 (not BMI) shift / rotate instructions are they aren't setup properly for RMW etc. Have you noticed anything else?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137841



More information about the llvm-commits mailing list