[Experiment] revert issue #26494 associated pulls #76986 and #79547#91719
Closed
shampoofactory wants to merge 0 commit intorust-lang:masterfrom
shampoofactory:master
Closed
[Experiment] revert issue #26494 associated pulls #76986 and #79547#91719shampoofactory wants to merge 0 commit intorust-lang:masterfrom shampoofactory:master
shampoofactory wants to merge 0 commit intorust-lang:masterfrom
shampoofactory:master
Conversation
Loading
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
15 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issue #26494 introduces a
P-highauto-vectorization regression as discussed here. This pull reverts those changes with a view to profiling the outcome. As a matter of expediency, I've ignored the 'array-equality.rs' test for now.If performance degrades we can take this option off the table. Otherwise, failing a timely fix to the underlying issue, a reversion could be discussed with concrete performance data.
If someone could kindly start a perf run.
Thank you.