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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to