Skip to content

Move call to log and fmaxf outside of inner loop used by LogSoftmax - #745

Merged
skottmckay merged 1 commit into
masterfrom
skottmckay/LogSoftMaxOptimization
Apr 1, 2019
Merged

Move call to log and fmaxf outside of inner loop used by LogSoftmax#745
skottmckay merged 1 commit into
masterfrom
skottmckay/LogSoftMaxOptimization

Conversation

@skottmckay

Copy link
Copy Markdown
Contributor

Value doesn't change in inner loop.

@skottmckay
skottmckay requested a review from a team as a code owner March 31, 2019 22:52
@skottmckay
skottmckay requested a review from gramalingam March 31, 2019 22:53
@skottmckay
skottmckay merged commit f4b47ad into master Apr 1, 2019
@skottmckay
skottmckay deleted the skottmckay/LogSoftMaxOptimization branch April 1, 2019 05:19
jnagi-intel pushed a commit to jnagi-intel/onnxruntime that referenced this pull request Jan 5, 2026
yuslepukhin pushed a commit that referenced this pull request Mar 17, 2026
…on pass (#745)

## Describe your changes

Fix output path for ranked models generated by transformer optimization
pass

Ranked model includes the model suffix name i.e. model_XX.onnx and the
pass should avoid appending it again.

## Checklist before requesting a review
- [ ] Add unit tests for this change.
- [ ] Make sure all tests can pass.
- [ ] Update documents if necessary.
- [ ] Lint and apply fixes to your code by running `lintrunner -a`
- [ ] Is this a user-facing change? If yes, give a description of this
change to be included in the release notes.

## (Optional) Issue link
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.

2 participants