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

[PoC] Add fast-math flags nnan and ninf for optimization #10

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

Conversation

augustoasilva
Copy link

@augustoasilva augustoasilva commented Jul 29, 2021

This PR can reduce up to 6.06% of the elapsed time without affecting the floating point operations result's precision.

@augustoasilva augustoasilva self-assigned this Jul 29, 2021
@augustoasilva augustoasilva changed the title Add fast-math flags nnan and ninf for optimization [PoC] Add fast-math flags nnan and ninf for optimization Jul 29, 2021
@github-actions
Copy link

Thanks for opening a pull request!

If this is not a minor PR. Could you open an issue for this pull request on JIRA? https://issues.apache.org/jira/browse/ARROW

Opening JIRAs ahead of time contributes to the Openness of the Apache Arrow project.

Then could you also rename pull request title in the following format?

ARROW-${JIRA_ID}: [${COMPONENT}] ${SUMMARY}

or

MINOR: [${COMPONENT}] ${SUMMARY}

See also:

ViniciusSouzaRoque pushed a commit that referenced this pull request Jul 13, 2022
…ation (apache#13579)

When locally building the `conda-cpp-valgrind` docker image I've realised we are prompting for confirmation:

```
#10 12.35 Confirm changes: [Y/n] 
#10 88.01  
```
We should use the -y flag when using mamba install:
```
  -y, --yes             Do not ask for confirmation.
```
With this change we are no longer seeing the `Confirm changes` message from mamba.

Authored-by: Raúl Cumplido <raulcumplido@gmail.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant