[mini] Cleanup MONO_MMAP_32BIT. (#16469)
commit0375acd7c8b1634a44477f2a561a10cd4d01858a
authorJay Krell <jaykrell@microsoft.com>
Wed, 28 Aug 2019 08:54:00 +0000 (28 01:54 -0700)
committerAlexander Köplinger <alex.koeplinger@outlook.com>
Wed, 28 Aug 2019 08:54:00 +0000 (28 10:54 +0200)
tree3e10082248c005003ea49e4dea9210eb0eabe6fa
parent162849f97202e8a24355fb77b01657b891436383
[mini] Cleanup MONO_MMAP_32BIT. (#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.
mono/mini/mini-arm.c
mono/mini/mini-mips.c
mono/mini/mini-ppc.c