search

dendibakh / perf-book

The book "Performance Analysis and Tuning on Modern CPU"
https://book.easyperf.net/perf_book
Creative Commons Zero v1.0 Universal
2.12k stars 156 forks source link

Update SIMD sections #21

Closed jan-wassenberg closed 1 year ago

dendibakh commented 1 year ago

Thank you @jan-wassenberg, I will review it in the upcoming days.

cf-natali commented 1 year ago

Might be worth adding a section on Intel AMX? It's not available in Sapphire Rapids and not that well known yet.

jan-wassenberg commented 1 year ago

Thank you @jan-wassenberg, for the write-up. There are no major rewrites, so I expect this PR to be merged quickly. Don't you mind if I will make cosmetic changes myself (maybe in a subsequent commit)?

Yes, please feel free to make cosmetic changes in a subsequent commit :)

jan-wassenberg commented 1 year ago

Might be worth adding a section on Intel AMX? It's not available in Sapphire Rapids and not that well known yet.

hm, from my perspective it seems a bit early to write about, personally I have not gathered much experience with it yet. There would also be SME and the various RISC-V extensions, plus POWER10. (BTW you mean it is available in SPR, right?)

dendibakh commented 1 year ago

Might be worth adding a section on Intel AMX? It's not available in Sapphire Rapids and not that well known yet.

hm, from my perspective it seems a bit early to write about, personally I have not gathered much experience with it yet. There would also be SME and the various RISC-V extensions, plus POWER10. (BTW you mean it is available in SPR, right?)

I haven't used AMX myself yet. :) Though it's on my TODO list. I have seen a short manual about programming with AMX intrinsics -- will try to find it. I wonder if compilers generate AMX code... I've heard AMX requires a special data layout (or transposition).

Let's not make a whole section about it, but I think mentioning matrix extensions is worth it.

jan-wassenberg commented 1 year ago

OK, I've added brief mention of the two AMX plus SME :)

dendibakh commented 1 year ago

Thanks @jan-wassenberg , I have done a few cosmetic updates and I'm ready to merge this PR. Let me know if you have anything else to change. If not, then we're done here.

jan-wassenberg commented 1 year ago

Very nice, thanks for making the changes. Looks good to me 👍