search

archspec / archspec-json

Other
20 stars 32 forks source link

Add Neoverse N2 #112

Closed paolotricerri closed 1 month ago

paolotricerri commented 2 months ago

fixes #97

This patch goes together with https://github.com/archspec/archspec/pull/170 . Similarly to that PR, this PR is intended to be a proposal on how to use the cpuid field to identify candidates on aarch64 architectures.

This patch introduces an example of neoverse_n2 example (in the test folder) and the neoverse_n2 in the microarchitectures.json. The addition in the json file would overwrite https://github.com/archspec/archspec-json/pull/105 but, as this PR is intended to be a proposal, I'll carefully manage it if PR#105 gets merged in first.

Warning: This patch has not removed the SM crypto features from the neoverse_v2 definition yet.

CC: @dslarm, @alalazo

paolotricerri commented 1 month ago

Hi @alalazo, I was wondering if you have had the time to have a look at the proposal?

Thanks P

alalazo commented 1 month ago

I assume this one supersedes #105, so let's continue working here. Can you rebase? Other than that this looks fine.

paolotricerri commented 1 month ago

HI @alalazo , have rebased and pushed to the branch.

Thanks