[PATCH] D130959: [X86] Add schedule module for Alderlake target

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon requested changes to this revision.
RKSimon added a comment.
This revision now requires changes to proceed.

This comes back to https://github.com/llvm/llvm-project/issues/56092 - I don't think we can have a single scheduler model called "alderlake" - the p cores and e cores behaviour are just too different. The models are used for analysis as much as scheduling.

As a first step it might be OK if we rename this model alderlake-p and use it by default for the -mcpu=alderlake target but do you intend to add a alderlake-e model as well?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130959