diff options
| author | Bernhard Urban <bernhard.urban@xamarin.com> | 2018-11-06 23:55:48 +0300 |
|---|---|---|
| committer | monojenkins <jo.shields+jenkins@xamarin.com> | 2018-11-06 23:55:48 +0300 |
| commit | 22143c7916d8f0b2095aeed4d1c4bf0c8cbfedc6 (patch) | |
| tree | 13470f3f5ef31af82222d91caed6922ad117b8a8 /sdks/builds/ios.mk | |
| parent | 5d5d8fc7ea64648a6ad5e3a90a055f77ac2f0660 (diff) | |
[icalls] remove DISABLE_POLICY_EVIDENCE (#11565)
[icalls] remove DISABLE_POLICY_EVIDENCE
Originally it was introduced with https://github.com/mono/mono/commit/db3591bb93dfe913c39ff73b95194effb88c880a . It seems like only the icall definitions were disabled for device runtimes (which aren't shipped there anyway), however the icalls themselves are _included_ in the device runtime. Note that the definitions are available to the AOT compiler.
Not sure what the rationale was behind this. Since the original change is very old, I think it just happened to be like that today. When running with the interpreter we need the icalls definitions at run-time available too (via the optional icall table that is linked in, so no additional cost for the fullAOT runtime due to this change).
Also remove `HANDLES_MAYBE` since it isn't needed anymore.
I think eventually we want to have `--enable-minimal=policy_evidence` that _also_ excludes the actual icalls, but this will need changes on the BCL side as well.
Diffstat (limited to 'sdks/builds/ios.mk')
| -rw-r--r-- | sdks/builds/ios.mk | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sdks/builds/ios.mk b/sdks/builds/ios.mk index cfe5d7a8981..0c5b6176403 100644 --- a/sdks/builds/ios.mk +++ b/sdks/builds/ios.mk @@ -70,7 +70,7 @@ _ios-$(1)_CPPFLAGS= \ -DMONOTOUCH=1 \ $$(ios-$(1)_SYSROOT) \ -arch $(3) \ - -DSMALL_CONFIG -DDISABLE_POLICY_EVIDENCE=1 -D_XOPEN_SOURCE -DHOST_IOS -DHAVE_LARGE_FILE_SUPPORT=1 \ + -DSMALL_CONFIG -D_XOPEN_SOURCE -DHOST_IOS -DHAVE_LARGE_FILE_SUPPORT=1 \ _ios-$(1)_LDFLAGS= \ -Wl,-no_weak_imports \ |