[PATCH] D92812: [X86] AMD Znver3 (Family 19H) Enablement

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 8 03:55:37 PST 2020


RKSimon added a comment.

This patch doesn't look like it is taking into account the existing (albeit limited) support for znver3 - it looks like a very bad merge imo.

It'd make much more sense to split this into several smaller incremental patches:

1 - cleanup any missing basic znver3 handling, building on the work that @bkramer has already done
2 - patches for the new INVLPGB/TLBSYNC/SNP instructions
3 - new znver3 scheduler model


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92812



More information about the llvm-commits mailing list