Это запах кода использовать std :: move для всех копий значений класса на всякий случай?

У меня есть функция, которая получает структуру конфигурации, которая на данный момент представляет собой просто структуру, содержащую массив

class ShmConfig {
public:
   std::int64_t shellShmSize[ShellId_Count];
};

И я передаю его другой функции в функции с std::move, потому что я хочу, чтобы все места кода, которые его копируют, не зависели от того факта, что его можно эффективно скопировать.

m_allocator.setup(std::move(config));

Если позже его можно будет более эффективно переместить (возможно, потому, что я добавлю к нему std::string), он автоматически станет более эффективным. Но clang-tidy не советует

std::move переменной 'config' тривиально копируемого типа 'ShmConfig' не имеет никакого эффекта; удалить std::move()

Не понимаю, почему это желательно. Есть ли недостаток в оборачивании move? Следует ли следовать этим советам только в том случае, если кто-то регулярно запускает clang-tidy, чтобы можно было обнаружить случаи, когда std::move будет иметь смысл позже, после добавления элементов данных?