Skip to content

Add cxx_flags to docs#455

Open
martinloretzzz wants to merge 1 commit intohuggingface:mainfrom
martinloretzzz:cxx_flags
Open

Add cxx_flags to docs#455
martinloretzzz wants to merge 1 commit intohuggingface:mainfrom
martinloretzzz:cxx_flags

Conversation

@martinloretzzz
Copy link
Copy Markdown

Fixes #454 (Opened the issue half an hour ago, found that it's already implemented, but missing in the docs).

When I tried to add cxx_flags with the underscore to the build.toml, I got the following error: Unknown field cxx_flags. Expected one of cxx-flags, depends, include, src, that means likely the cuda_flags and sycl_flags should also be cuda-flags sycl-flags, should I also update this in the docs?

@danieldk
Copy link
Copy Markdown
Member

Thanks, great catch! Is should indeed also be cuda-flags and sycl-flags, would be great if you could add them to this PR.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add option to specify compiler flags on the CPU (like -ffast-math)

2 participants