From 0db598e48d68d267c13b8213b333f8ec2b13d28f Mon Sep 17 00:00:00 2001 From: Brandon Potter Date: Mon, 11 Mar 2019 14:08:17 -0400 Subject: [PATCH] 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 Tested-by: kokoro Reviewed-by: Jason Lowe-Power --- src/sim/process.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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 nfds = np->fds; for (int tgt_fd = 0; tgt_fd < fds->getSize(); tgt_fd++) { - std::shared_ptr nfds = np->fds; std::shared_ptr this_fde = (*fds)[tgt_fd]; if (!this_fde) { nfds->setFDEntry(tgt_fd, nullptr); -- 2.30.2