Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Some lscpu flags need an alias #101

Closed
alexKrauseTUD opened this issue Jun 27, 2024 · 1 comment · Fixed by #103
Closed

Some lscpu flags need an alias #101

alexKrauseTUD opened this issue Jun 27, 2024 · 1 comment · Fixed by #103
Assignees
@JPietrzykTUD
Labels
bug Something isn't working

Comments

@alexKrauseTUD
Copy link
Collaborator

When the TSL is generated, the required compiler arguments are derived from the lscpu list; either trhough py-cpuinfo or lscpu.

However, some flags have no direct mapping to compiler flags, especially avx512 flags, e.g.

lscpu g++/clang
avx512_fp16 -mavx512fp16
avx512_vpopcntdq -mavx512vpopcntdq
avx512_vbmi2 -mavx512vbmi2

Maybe we can conventionally just remove the underscores, but this might not be true for all flags.
@alexKrauseTUD alexKrauseTUD added the bug Something isn't working label Jun 27, 2024
@JPietrzykTUD JPietrzykTUD linked a pull request Jul 9, 2024 that will close this issue
Issues 99 102 #103
Merged
JPietrzykTUD added a commit that referenced this issue Jul 9, 2024
@JPietrzykTUD
Added architecture mappings for avx512
1e65833 
See issue #101
@JPietrzykTUD
Copy link
Collaborator

Fixed with #103

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JPietrzykTUD JPietrzykTUD

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Issues 99 102 db-tu-dresden/TSL
2 participants
@JPietrzykTUD @alexKrauseTUD