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

[Haohai, Wen via Phabricator via llvm-commits]

HaohaiWen added a comment.

> All I'm actually requiring is that the instregex that end in "_Int$" are replaced with (_Int)?$

I have converted something like instregex "^SHA1MSG2rr$" to nstrs SHA1MSG2rr.

_Int version is not CodeGenOnly but the same prefix instruction without _Int is usually CodeGenOnly.
As I explained in https://github.com/llvm/llvm-project/issues/58792. We have some problem to auto gen schedule description for CodeGenOnly instruction due to wrong predicates issue. Therefore, non-Int version opcode will actually use scheduling info for its default schedreadwrite.
I know this is a bug and I'm trying to fix it.

@RKSimon , If you need a quick fix for _Int issue, please feel free to modify this schedule model. I will auto gen a new model after solving this problem.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137841