diff options
| author | Brandon Potter <brandon.potter@amd.com> | 2019-03-11 14:08:17 -0400 |
|---|---|---|
| committer | Brandon Potter <Brandon.Potter@amd.com> | 2019-08-01 20:31:33 +0000 |
| commit | 0db598e48d68d267c13b8213b333f8ec2b13d28f (patch) | |
| tree | 563a25fb16e8ccf6e3af2ac5d209d078abadff02 /src/sim | |
| parent | 9afe480c404543a0f7fe59b3abe5bd3d50367c5c (diff) | |
| download | gem5-0db598e48d68d267c13b8213b333f8ec2b13d28f.tar.xz | |
sim-se: small performance optimization
A local variable was being set inside a loop when it should
have been set outside the loop. This changeset moves the
variable to the appropriate place.
Change-Id: If7655b501bd819c39d35dea4c316b4b9ed3173a2
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/17108
Maintainer: Brandon Potter <Brandon.Potter@amd.com>
Tested-by: kokoro <noreply+kokoro@google.com>
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Diffstat (limited to 'src/sim')
| -rw-r--r-- | src/sim/process.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/sim/process.cc b/src/sim/process.cc index f974fa5ba..3a39dbe50 100644 --- a/src/sim/process.cc +++ b/src/sim/process.cc @@ -199,8 +199,8 @@ Process::clone(ThreadContext *otc, ThreadContext *ntc, * host file descriptors are also dup'd so that the flags for the * host file descriptor is independent of the other process. */ + std::shared_ptr<FDArray> nfds = np->fds; for (int tgt_fd = 0; tgt_fd < fds->getSize(); tgt_fd++) { - std::shared_ptr<FDArray> nfds = np->fds; std::shared_ptr<FDEntry> this_fde = (*fds)[tgt_fd]; if (!this_fde) { nfds->setFDEntry(tgt_fd, nullptr); |