Merge branch 'cache-get-workerid' into 'master'

Re-use Runtime::getWorkerId() when possible

See merge request !44

emper/MemoryManager.hpp

@@ -40,12 +40,13 @@ class MemoryManager {
 		if (likely(poped)) return memory;
 
 		// Fallback to memory-stealing.
+		const workerid_t myWorkerId = Runtime::getWorkerId();
 		const workerid_t startWorkerId = Runtime::rand() % workerCount;
 		for (workerid_t i = 0; i < workerCount; ++i) {
 			workerid_t victim = (startWorkerId + i) % workerCount;
 
 			// Don't steal from ourselves.
-			if (unlikely(victim == Runtime::getWorkerId())) continue;
+			if (unlikely(victim == myWorkerId)) continue;
 
 			poped = queues[victim]->popTop(&memory);
 			if (poped) return memory;

emper/strategies/laws/LawsScheduler.cpp

@@ -102,7 +102,7 @@ auto LawsScheduler::nextFiber() -> Fiber* {
 
 	poped = queues[victim]->popTop(&fiber);
 	if (poped) {
-		if (emper::STATS) {
+		if constexpr (emper::STATS) {
 			auto flag = static_cast<unsigned int>(LawsStrategy::FiberSource::stolen);
 			fiber->setFlag(flag);
 		}

emper/strategies/ws/WsScheduler.cpp

@@ -54,6 +54,7 @@ auto WsScheduler::nextFiber() -> Fiber* {
 		return fiber;
 	}
 
+	const workerid_t myWorkerId = Runtime::getWorkerId();
 	const workerid_t workerCount = runtime.getWorkerCount();
 	workerid_t startWorkerId = Runtime::rand() % workerCount;
 	// TODO: See how reducing the loop bound affects things.
@@ -61,7 +62,7 @@ auto WsScheduler::nextFiber() -> Fiber* {
 		workerid_t victim = (startWorkerId + i) % workerCount;
 
 		// Don't steal from ourselves.
-		if (unlikely(victim == Runtime::getWorkerId())) continue;
+		if (unlikely(victim == myWorkerId)) continue;
 
 		poped = queues[victim]->popTop(&fiber);
 		if (poped) {