Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Avoid cloning the stack on every
push_ctxt
call in trans
Rewrite the use of TLS variable for `push_ctxt` so that it uses a ~[] instead of a @~[]. Before it cloned the whole vector on each push and pop, which is unnecessary.
- Loading branch information
blake2-ppc
committed
Oct 4, 2013
1 parent
9344e2a
commit 87294c2
Showing
1 changed file
with
11 additions
and
11 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
87294c2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
saw approval from alexcrichton
at https://github.com/blake2-ppc/rust/commit/87294c23baacaee17a13d2f7316fffc76239cff5
87294c2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
merging blake2-ppc/rust/trans-no-push-ctxt-clone = 87294c2 into auto
87294c2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
blake2-ppc/rust/trans-no-push-ctxt-clone = 87294c2 merged ok, testing candidate = 8cb3426
87294c2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/1842
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/1846
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-c/builds/952
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/951
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/1860
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-c/builds/952
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/952
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/1861
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-c/builds/952
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/952
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android/builds/1032
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/1845
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-c/builds/951
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/952
success: http://buildbot.rust-lang.org/builders/auto-bsd-64-opt/builds/1630
87294c2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fast-forwarding master to auto = 8cb3426