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.

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

crystal: optimize the build command even further #380

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

renich
Copy link

@renich renich commented Mar 23, 2023

It's suggested that you build with: --release --no-debug in order to optimize for performance.

Also, the documentation states the following about --mcpu=:

               --mcpu CPU
                         Specify a specific CPU to generate code for. This will pass a -mcpu flag to LLVM, and is only intended to be used for cross-compilation. For
                         a list of available CPUs, invoke "llvm-as < /dev/null | llc -march=xyz -mcpu=help".  Passing --mcpu native will pass the host CPU name to
                         tune performance for the host.

If we are to use it, then, at the very least, use --mcpu=native.

It's suggested that you build with: `--release --no-debug` in order to optimize for performance. 

Also, the documentation states the following about `--mcpu=`:

```
               --mcpu CPU
                         Specify a specific CPU to generate code for. This will pass a -mcpu flag to LLVM, and is only intended to be used for cross-compilation. For
                         a list of available CPUs, invoke "llvm-as < /dev/null | llc -march=xyz -mcpu=help".  Passing --mcpu native will pass the host CPU name to
                         tune performance for the host.
```

If we are to use it, then, at the very least, use `--mcpu=native`.
@hanabi1224
Copy link
Owner

Thanks, but it's intended to use --mcpu=broadwell for all launguages

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.

None yet

2 participants