Cleanup trampoline size checks. (mono/mono#15624)
authorJay Krell <jaykrell@microsoft.com>
Thu, 29 Aug 2019 20:16:14 +0000 (13:16 -0700)
committermonojenkins <jo.shields+jenkins@xamarin.com>
Thu, 29 Aug 2019 20:16:14 +0000 (16:16 -0400)
commitb30897cc39ffc50b4bf25e962469164b419f37eb
tree4bcb1ed3e276c103ad5d056156f78444b32d2fde
parent30887533f3fda32b56d9b19a5b7100da9e3788f7
Cleanup trampoline size checks. (mono/mono#15624)

Cleanup trampoline size checks.

- named `int` everywhere
- `assert` everywhere
- assert `<=` everywhere instead of `<`
- `const` almost everywhere
Names not consistent, following existing code (`size`, `buf_len`, `kMaxCode`.)

Only touching x86/amd64, granted.

Could be better -- make them two pass and avoid the presizing.

Commit migrated from https://github.com/mono/mono/commit/5390b796dd9c16eaded61c406a4cea9c66ea8905
src/mono/mono/mini/exceptions-amd64.c
src/mono/mono/mini/exceptions-x86.c
src/mono/mono/mini/mini-amd64.c
src/mono/mono/mini/mini-x86.c
src/mono/mono/mini/tramp-amd64-gsharedvt.c
src/mono/mono/mini/tramp-amd64.c
src/mono/mono/mini/tramp-x86-gsharedvt.c
src/mono/mono/mini/tramp-x86.c