[PATCH] D139073: [X86] AMD Zen 4 Initial enablement

2022-12-17 Thread Ganesh Gopalasubramanian via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG1f057e365f1f: [X86] AMD Zen 4 Initial enablement (authored by GGanesh). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://r

[PATCH] D139073: [X86] AMD Zen 4 Initial enablement

2022-12-15 Thread Simon Pilgrim via Phabricator via cfe-commits
RKSimon accepted this revision. RKSimon added a comment. cheers Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D139073/new/ https://reviews.llvm.org/D139073 ___ cfe-commits mailing list cfe-commits@lists.l

[PATCH] D139073: [X86] AMD Zen 4 Initial enablement

2022-12-15 Thread Ganesh Gopalasubramanian via Phabricator via cfe-commits
GGanesh updated this revision to Diff 483141. GGanesh added a comment. Fixed the tests tools/llvm-mca/X86/cpus.s tools/llvm-mca/X86/read-after-ld-1.s tools/llvm-mca/X86/register-file-statistics.s tools/llvm-mca/X86/scheduler-queue-usage.s Repository: rG LLVM Github Monorepo CHANGES SINCE LAS

[PATCH] D139073: [X86] AMD Zen 4 Initial enablement

2022-12-15 Thread Ganesh Gopalasubramanian via Phabricator via cfe-commits
GGanesh added a comment. In D139073#3997418 , @RKSimon wrote: > Thanks @GGanesh - LGTM with one minor - the orphan ZNVER4 checks from the > llvm-mca tests need removing Thank you! I left them intentionally as this is a stop-gap patch. Agreed! Will rem

[PATCH] D139073: [X86] AMD Zen 4 Initial enablement

2022-12-15 Thread Simon Pilgrim via Phabricator via cfe-commits
RKSimon accepted this revision. RKSimon added a comment. This revision is now accepted and ready to land. Thanks @GGanesh - LGTM with one minor - the orphan ZNVER4 checks from the llvm-mca tests need removing Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.ll

[PATCH] D139073: [X86] AMD Zen 4 Initial enablement

2022-12-15 Thread Ganesh Gopalasubramanian via Phabricator via cfe-commits
GGanesh updated this revision to Diff 483092. GGanesh added a comment. Update to use 'no' processor model. Fixed the below tests tools/llvm-mca/X86/cpus.s tools/llvm-mca/X86/read-after-ld-1.s tools/llvm-mca/X86/register-file-statistics.s tools/llvm-mca/X86/scheduler-queue-usage.s Addressed commen

[PATCH] D139073: [X86] AMD Zen 4 Initial enablement

2022-12-14 Thread Ganesh Gopalasubramanian via Phabricator via cfe-commits
GGanesh added a comment. In D139073#3995741 , @RKSimon wrote: > @GGanesh reverse-ping @RKSimon Thanks a lot! We were trying to get the libpfm patch posted and subsequently enabling the zen4 scheduler model. Probably a week or two from there. The intel

[PATCH] D139073: [X86] AMD Zen 4 Initial enablement

2022-12-14 Thread Simon Pilgrim via Phabricator via cfe-commits
RKSimon added a comment. @GGanesh reverse-ping Comment at: llvm/lib/Target/X86/X86.td:1619 ProcessorFeatures.ZN3Tuning>; +def : ProcModel<"znver4", Znver3Model, ProcessorFeatures.ZN4Features, +ProcessorFeatures.ZN4Tuning>; RKSim

[PATCH] D139073: [X86] AMD Zen 4 Initial enablement

2022-12-01 Thread Simon Pilgrim via Phabricator via cfe-commits
RKSimon added a comment. Thank you @gganesh I thought you'd forgotten about us :) Please add znver4 test coverage to: llvm/test/CodeGen/X86/cpus-amd.ll llvm/test/CodeGen/X86/slow-unaligned-mem.ll Comment at: llvm/lib/Target/X86/X86.td:1619 ProcessorFeatures.Z

[PATCH] D139073: [X86] AMD Zen 4 Initial enablement

2022-11-30 Thread Ganesh Gopalasubramanian via Phabricator via cfe-commits
GGanesh created this revision. GGanesh added reviewers: RKSimon, craig.topper, andreadb. Herald added subscribers: Enna1, StephenFan, pengfei, gbedwell, javed.absar, hiraditya. Herald added a project: All. GGanesh requested review of this revision. Herald added projects: clang, Sanitizers, LLVM. H