IIRC @qwattash chose vm_pointer_t as an easy way to keep track changes to the VM system vs converting to uintptr_t. I believe @bsdjhb thinks we should convert some more of them to void * where we need to preserve provenance, but don't need to be manipulated. Should we upstream vm_pointer_t at all or should we be migrating the VM system away from mach types as part of upstreaming?