[mini] Cleanup MONO_MMAP_32BIT. (mono/mono#16469)
authorJay Krell <jaykrell@microsoft.com>
Wed, 28 Aug 2019 08:54:00 +0000 (01:54 -0700)
committerAlexander Köplinger <alex.koeplinger@outlook.com>
Wed, 28 Aug 2019 08:54:00 +0000 (10:54 +0200)
commita1b5d493d0aa9bd14a5be253f1db2946719f9206
treeda6d17e471bf8ffb90d0db61c0ad9ea203b994c9
parentff48d35115cfbbc0856a178ac6c731aec2192e7b
[mini] Cleanup MONO_MMAP_32BIT. (mono/mono#16469)

- It _presumably_ means nothing on 32bit ARM and 32bit MIPS (was there mono/mips64?).
 - Leaving only PowerPC32 and PowerPC64 (in this vicinity).
   - It presumably means nothing on PowerPC32.
   - Leaving PowerPC64 as the only user in this vicinity -- is it needed?
- Notice this is varyingly absent on x86, arm64, amd64, s390x.
  Subject of another PR maybe.

Perhaps this was to enable the debugger to be 32bit against a 64bit target
on a biarch system, so it could do a cross-process manipulation of these pages?
That makes some sense on Windows at least.
(32bit debugger cannot debug 64bit target as of Vista, but that is using the Win32
"hard" debugger API. Mono could do otherwise).

There remains amd64 mono-codeman.c define ARCH_MAP_FLAGS MONO_MMAP_32BIT left alone.

Commit migrated from https://github.com/mono/mono/commit/0375acd7c8b1634a44477f2a561a10cd4d01858a
src/mono/mono/mini/mini-arm.c
src/mono/mono/mini/mini-mips.c
src/mono/mono/mini/mini-ppc.c