Правильна ли моя реализация шаблона блокировки с двойной проверкой?

Пример в книге Мейерса Эффективный Современный C ++, Пункт 16

в кешировании класса дорогой для вычисления int, вы можете попытаться использовать
пара доступных std :: atomic вместо мьютекса:

class Widget {
public:
int magicValue() const {
if (cachedValid) {
return cachedValue;
} else {
auto val1 = expensiveComputation1();
auto val2 = expensiveComputation2();

cachedValue = va1 + val2;
cacheValid = true;
return cachedValue;
}
}
private:
mutable std::atomic<bool> cacheValid { false };
mutable std::atomic<int> cachedValue;
};

Это будет работать, но иногда это будет работать намного тяжелее, чем это
should.Consider: поток вызывает Widget :: magicValue, видит cacheValid как
false, выполняет два дорогих вычисления и присваивает их сумму
к кешированию На этом этапе второй поток calss
Widget :: magicValue, также видит cacheValid как false и, таким образом, переносит
из тех же дорогих вычислений, что первый поток имеет только
законченный.

Затем он дает решение с мьютексом:

class Widget {
public:
int magicValue() const {
std::lock_guard<std::mutex> guard(m);
if (cacheValid) {
return cachedValue;
} else {
auto val1 = expensiveComputation1();
auto val2 = expensiveComputation2();

cachedValue = va1 + val2;
cacheValid = true;
return cachedValue;
}
}
private:
mutable std::mutex m;
mutable bool cacheValid { false };
mutable int cachedValue;
};

Но я думаю, что решение не так эффективно, я считаю, чтобы объединить мьютекс и атом, чтобы создать Двойная проверка шаблона блокировки как ниже.

class Widget {
public:
int magicValue() const {
if (!cacheValid)  {
std::lock_guard<std::mutex> guard(m);
if (!cacheValid) {
auto val1 = expensiveComputation1();
auto val2 = expensiveComputation2();

cachedValue = va1 + val2;
cacheValid = true;
}
}
return cachedValue;
}
private:
mutable std::mutex m;
mutable std::atomic<bool> cacheValid { false };
mutable std::atomic<int> cachedValue;
};

Потому что я новичок в многопоточном программировании, поэтому я хочу знать:

• Правильный ли мой код?
• Это производительность лучше?

РЕДАКТИРОВАТЬ:

Исправлен код.if (! cachedValue) -> if (! cacheValid)