From 22143c7916d8f0b2095aeed4d1c4bf0c8cbfedc6 Mon Sep 17 00:00:00 2001 From: Bernhard Urban Date: Tue, 6 Nov 2018 21:55:48 +0100 Subject: [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. --- sdks/builds/ios.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'sdks/builds/ios.mk') 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 \ -- cgit v1.2.3