Common/PointerWrap: Remove DoPOD

[Pokechu22]

This was added in 385d8e2, but became somewhat redundant with Do in 4c7bbd9, and completely redundant now that std::is_trivially_copyable_v is well-supported.

[Pokechu22]

Note: this is mostly equivalent (as long as num_blocks is less than INT_MAX, and calling Do n times is the same as calling a larger DoArray, which should be the case for a trivially copyable type):

dolphin/Source/Core/Common/ChunkFile.h

Lines 125 to 129 in de5a98a

	template <typename T>
	void Do(std::vector<T>& x)
	{
	DoContiguousContainer(x);
	}

dolphin/Source/Core/Common/ChunkFile.h

Lines 319 to 328 in de5a98a

	template <typename T>
	void DoContiguousContainer(T& container)
	{
	u32 size = static_cast<u32>(container.size());
	Do(size);
	container.resize(size);
	if (size > 0)
	DoArray(&container[0], size);
	}

dolphin/Source/Core/Common/ChunkFile.h

Lines 198 to 202 in de5a98a

	template <typename T, typename std::enable_if_t<std::is_trivially_copyable_v<T>, int> = 0>
	void DoArray(T* x, u32 count)
	{
	DoVoid(x, count * sizeof(T));
	}

[AdmiralCurtiss]

While I think you're right that this is equivalent, I think I'd feel safer if you bumped up the savestate version anyway -- #11125 is going to bump it too so it's no big deal to have a break here.

[Pokechu22]

Done. I also changed the similar manual std::vector handling in GCMemcardDirectory.cpp.